Skip to content

Homepage updates - #16861

Merged
maureenlholland merged 15 commits into
mainfrom
homepage-updates
Nov 19, 2025
Merged

Homepage updates#16861
maureenlholland merged 15 commits into
mainfrom
homepage-updates

Conversation

@wen-2018

@wen-2018 wen-2018 commented Nov 13, 2025

Copy link
Copy Markdown
Collaborator

If this changeset needs to go into the FXC codebase, please add the WMO and FXC label.

One-line summary

Significant changes and points to review

Issue / Bugzilla link

#16797
WT-328

Testing

http://localhost:8000/en-US/

@codecov

codecov Bot commented Nov 13, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.00%. Comparing base (19020a2) to head (8ddcf91).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #16861   +/-   ##
=======================================
  Coverage   80.00%   80.00%           
=======================================
  Files         161      161           
  Lines        8778     8778           
=======================================
  Hits         7023     7023           
  Misses       1755     1755           

☔ 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.

@wen-2018 wen-2018 added Do Not Merge Needs Review Awaiting code review labels Nov 13, 2025
@maureenlholland maureenlholland self-assigned this Nov 18, 2025

@maureenlholland maureenlholland left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

So many images! 🖼️ Nice work wrangling all the changes

Blocking:

  • condition referring to now-outdated FTL string
  • rogue "Raffi headshot" text
  • https://archive.is/kj2u7 is failing to load for me

Potentially blocking:

  • The pink cloud images get quite large. Maybe they need another round of optimization or a different image format?
  • I'm not sure how we want to handle single-language third party content on a multi-language page

Comment thread bedrock/base/templates/includes/protocol/footer/footer-refresh.html Outdated
Comment thread l10n/en/mozorg/home-m24.ftl Outdated
Comment thread l10n/en/mozorg/home-m24.ftl Outdated
Comment thread media/img/icons/m24-small/arrow-right.svg
Comment thread bedrock/mozorg/templates/mozorg/home/includes/m24/data-collective.html Outdated
Comment thread media/img/home/2024/ai-gallery/pink-cloud-mobile-1450.png
Comment thread bedrock/mozorg/templates/mozorg/home/includes/m24/media-springboard.html Outdated
Comment thread bedrock/mozorg/templates/mozorg/home/includes/m24/ai-gallery.html Outdated
Comment thread media/css/m24/fox.scss
@wen-2018 wen-2018 removed Do Not Merge Needs Review Awaiting code review labels Nov 19, 2025
@wen-2018
wen-2018 marked this pull request as ready for review November 19, 2025 14:53
@wen-2018
wen-2018 requested a review from a team as a code owner November 19, 2025 14:53
@maureenlholland
maureenlholland self-requested a review November 19, 2025 16:21

@maureenlholland maureenlholland left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM r+ 🚢

@maureenlholland
maureenlholland merged commit b6d9193 into main Nov 19, 2025
5 checks passed
@maureenlholland
maureenlholland deleted the homepage-updates branch November 19, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants