Skip to content

Refresh solutions/telco/5g-core page - #2941

Open
mozsarmate wants to merge 1 commit into
canonical:mainfrom
mozsarmate:WD-38576-dev-refresh-canonical-com-solutions-telco-5-g-core
Open

mozsarmate wants to merge 1 commit into
canonical:mainfrom
mozsarmate:WD-38576-dev-refresh-canonical-com-solutions-telco-5-g-core

Conversation

@mozsarmate

@mozsarmate mozsarmate commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Done

Refreshed page content and structure based on the copy Docs and Figma.

QA

  • Open the DEMO
  • Alternatively, check out this feature branch
  • Run the site using the command dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8002/
  • Run through the following QA steps
  • [List additional steps to QA the new features or prove the bug has been resolved]

Issue / 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

Copilot AI lite review requested due to automatic review settings September 10, 2026 13:11
@webteam-app

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.83%. Comparing base (08b3a3d) to head (a47375c).
⚠️ Report is 4 commits behind head on main.

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     
Flag Coverage Δ
python 75.83% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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 class attribute 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 class attribute 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.

Comment thread templates/solutions/telco/5g-core.html Outdated
Comment thread templates/solutions/telco/5g-core.html Outdated
Comment thread templates/solutions/telco/5g-core.html Outdated
@mozsarmate

Copy link
Copy Markdown
Contributor Author

Hey @kuidja-tchatat and @eliman11,
I prepared the dev part of this page refresh. Can you please have a look on the design & UX, if you have a moment. Thanks!

@eliman11

eliman11 commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

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!
Screenshot 2026-09-11 at 12 35 40

@mozsarmate
mozsarmate force-pushed the WD-38576-dev-refresh-canonical-com-solutions-telco-5-g-core branch from 3adfef3 to ea182d5 Compare September 11, 2026 12:33
@mozsarmate

mozsarmate commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Hello @eliman11 , @kuidja-tchatat,
I swapped the quote section, corrected the alt texts and hid the image (of our stack) for mobile screens. Can you recheck, please? Thanks

@eliman11

Copy link
Copy Markdown
Collaborator

LGTM UX wise thanks so much!

@kuidja-tchatat

kuidja-tchatat commented Sep 11, 2026

Copy link
Copy Markdown

Hey @mozsarmate We’ve talked quite a bit on Mattermost, but I’m adding the comments here for history:

  • Change the code section → done
  • To stay consistent with the Telco bubble, use the "Learn more" section from the former page and do a copy update, using the links from the Figma file and copy doc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants