Refresh solutions/telco/5g-core page - #2941
mozsarmate wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2941 +/- ##
==========================================
- Coverage 75.84% 75.83% -0.02%
==========================================
Files 20 20
Lines 2596 2599 +3
==========================================
+ Hits 1969 1971 +2
- Misses 627 628 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
🟡 Changes recommended
The updated template introduces a few concrete markup/typo issues (e.g., “MASS” vs “MAAS” and empty/whitespace class attributes) that should be corrected before approval.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR refreshes the /solutions/telco/5g-core marketing page template, updating page structure and copy to align with the latest content direction for Canonical’s telco/5G Core messaging.
Changes:
- Updates the hero, headings, and meta content to reflect refreshed positioning and messaging.
- Restructures multiple sections into new grid/pricing-block layouts and adds a new logo-section block.
- Refreshes supporting links/resources (case study CTA text, webinars, suggested reading) and updates embedded media/content blocks.
File summaries
| File | Description |
|---|---|
templates/solutions/telco/5g-core.html |
Refreshed 5G Core solutions page layout, content blocks, and supporting links/media. |
Review details
Suppressed comments (3)
templates/solutions/telco/5g-core.html:418
- Remove the trailing whitespace in the
classattribute value;class="p-section "adds an extra (empty) class token and is easy to miss during future edits.
<div class="p-section ">
templates/solutions/telco/5g-core.html:488
- Remove the trailing whitespace in the
classattribute value;class="p-section "adds an extra (empty) class token and is easy to miss during future edits.
<div class="p-section " id="2-blocks-private-5g">
templates/solutions/telco/5g-core.html:562
class=""is an empty attribute and should be removed to keep the markup clean.
<h2 class="">Canonical's 5G Core infrastructure stack</h2>
- Files reviewed: 1/1 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
a3b4e64 to
c0b8833
Compare
c0b8833 to
3adfef3
Compare
|
Hey @kuidja-tchatat and @eliman11, |
|
Thanks @mozsarmate! A couple minor comments -
Edit: Sorry one more ask - can you please hide this entire section on mobile, as the text becomes illegible at that size. Thanks! |
3adfef3 to
ea182d5
Compare
|
Hello @eliman11 , @kuidja-tchatat, |
|
LGTM UX wise thanks so much! |
|
Hey @mozsarmate We’ve talked quite a bit on Mattermost, but I’m adding the comments here for history:
|
ea182d5 to
a47375c
Compare

Done
Refreshed page content and structure based on the copy Docs and Figma.
QA
dotrunIssue / Card
Jira ticket: https://warthogs.atlassian.net/browse/WD-38576
Copy Doc URL: https://docs.google.com/document/d/1h5A07aTGTZl5VfoiX2UwIpb-bHtFr7gBVUejN-uAf-Q/edit?tab=t.0
Figma file : https://www.figma.com/design/brgzRViEncgKKD4KzVnyXy/canonical.com-solutions---Sites?node-id=5592-13132&t=9UkBejVF8N9pzrwW-11