Skip to content

feat(WD-35846): dev create canonical com ceph - #2511

Merged
Skazitron merged 20 commits into
mainfrom
WD-35846-dev-create-canonical-com-ceph
May 6, 2026
Merged

feat(WD-35846): dev create canonical com ceph#2511
Skazitron merged 20 commits into
mainfrom
WD-35846-dev-create-canonical-com-ceph

Conversation

@Skazitron

@Skazitron Skazitron commented May 6, 2026

Copy link
Copy Markdown
Contributor

Done

  • Created /ceph page
  • Added contact form
  • Added navbar

QA

  • Open /ceph
  • Ensure page design matches figma doc
  • Ensure it matches the copydoc
  • Ensure contact form button works as expected (opens modal and displays success banner when submitted)
  • Ensure contact form matches the contact form on ubuntu.com/ceph

Issue / Card

https://warthogs.atlassian.net/browse/WD-35846
https://warthogs.atlassian.net/browse/WD-36247

@webteam-app

Copy link
Copy Markdown

@codecov

codecov Bot commented May 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.96%. Comparing base (017f51f) to head (1de2fe9).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2511   +/-   ##
=======================================
  Coverage   78.96%   78.96%           
=======================================
  Files          16       16           
  Lines        1930     1930           
=======================================
  Hits         1524     1524           
  Misses        406      406           
Flag Coverage Δ
python 78.96% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.

@Skazitron Skazitron changed the title Wd 35846 dev create canonical com ceph feat(Wd-35846): dev create canonical com ceph May 6, 2026
@Skazitron Skazitron changed the title feat(Wd-35846): dev create canonical com ceph feat(WD-35846): dev create canonical com ceph May 6, 2026
@mattea-turic

mattea-turic commented May 6, 2026

Copy link
Copy Markdown
Collaborator

Thanks @Skazitron !

Comments:
Hero:

  • Pls add image highlight class to the img container
  • And also apply .row--50-50 for 50/50 split on med breakpoint

"Canonical Ceph: Proven and trusted":

  • Is there a way to avoid the wrapping of the heading text on medium and small screens?

Screenshot 2026-05-06 at 2 01 19 PM

"Notable ceph users":

"More choice for your infrastructure":

  • Same comment to add the image highlight class

"Why choose Canonical Ceph?":

  • Sorry to go back on this, but just wanna check with @Sophie-32 or @eliman11 – do you guys think it looks better with, or without the <br>? I don't really like either tbh:

With break:
Screenshot 2026-05-06 at 1 54 37 PM

Without break:
Screenshot 2026-05-06 at 1 53 02 PM

"Learn more about Canonical Ceph":

  • The item headings here should be h5 styling
  • Pls remove the shallow from the last item ("AI storage with Ceph")

@mattea-turic

Copy link
Copy Markdown
Collaborator

@Skazitron

LGTM, thanks!

@Stefan3002

Stefan3002 commented May 6, 2026

Copy link
Copy Markdown
Contributor

@Skazitron in the rendered page, inside the demo, the blog posts get this kind of link:

Blog image

<a href="https://https://canonical.com/blog/microceph-why-its-the-superior-minio-alternative"><img src="https://res.cloudinary.com/canonical/image/fetch/f_auto,q_auto,fl_sanitize,w_4000/https://ubuntu.com/wp-content/uploads/63cc/MicroCeph-minio-blog-1.png" alt="Blog image" loading="lazy" width="4000" height="2250"></a>

It has double https:// resulting in a navigation error.

@Stefan3002
Stefan3002 self-requested a review May 6, 2026 14:45

@Stefan3002 Stefan3002 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.

LGTM, great job!

@Skazitron
Skazitron merged commit b2c3703 into main May 6, 2026
12 of 13 checks passed
@Skazitron
Skazitron deleted the WD-35846-dev-create-canonical-com-ceph branch May 6, 2026 14:58
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