Skip to content

Website: update homepage hero and header styles#45198

Merged
eashaw merged 7 commits into
mainfrom
2026-05-11-update-homepage-hero
May 11, 2026
Merged

Website: update homepage hero and header styles#45198
eashaw merged 7 commits into
mainfrom
2026-05-11-update-homepage-hero

Conversation

@eashaw
Copy link
Copy Markdown
Contributor

@eashaw eashaw commented May 11, 2026

Closes: #45126
Closes: #45128

Changes:

  • Updated the styles and layout of the homepage hero to match the latest wireframes
  • Removed the box-shadow from the website masthead and added a bottom border.
  • Added a new variable to colors.less @core-fleet-black-5

Summary by CodeRabbit

  • Style
    • Redesigned homepage hero layout and typography for improved responsive behavior across desktop, tablet, and mobile.
    • Reorganized quote and statistics into a unified responsive block with updated spacing, padding, and stacking.
    • Adjusted hero background sizing and hero-area height/padding for small screens.
    • Updated hero subtitle wording for clarity.
    • Refined header bottom color and removed header shadow.
    • Added a subtle new color token to the site palette.

Review Change Stack

Copilot AI review requested due to automatic review settings May 11, 2026 22:21
@fleet-release fleet-release added the ~ga4-annotation This label is automatically applied to certain website changes label May 11, 2026
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

fleet-release
fleet-release previously approved these changes May 11, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 7f5b229b-e150-40cf-bddd-c4edca292756

📥 Commits

Reviewing files that changed from the base of the PR and between 7dac043 and 8aeb48e.

📒 Files selected for processing (1)
  • website/assets/styles/pages/homepage.less
🚧 Files skipped from review as they are similar to previous changes (1)
  • website/assets/styles/pages/homepage.less

Walkthrough

This PR adds a new LESS color variable @core-fleet-black-5 (#F2F2F5) and applies it to the masthead border-bottom (the masthead box-shadow is commented out). The homepage markup moves the hero quote into a new quote-and-statistics section and updates the hero subtitle punctuation. Styles for the hero (spacing, quote/profile typography, statistics layout) are refactored under the new wrapper and adjusted across breakpoints, including changes to hero background-image sizing and mobile/desktop heights.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description covers the main changes and references the related issues (#45126, #45128) but does not follow the repository's PR template structure with checklists for validation, testing, and database considerations. Provide a complete description following the template, including relevant checklist items for manual QA, automated tests, and any applicable sections from the submission checklist.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: updating homepage hero layout and header styles, which matches the file modifications across layout.less, colors.less, homepage.less, and homepage.ejs.
Linked Issues check ✅ Passed All coding requirements from linked issues are met: #45126 implemented the new @core-fleet-black-5 color variable and applied it to the header border, and #45128 restructured the homepage hero layout and relocated the quote to the social proof section.
Out of Scope Changes check ✅ Passed All changes are within scope: color variable addition, homepage hero and header styling updates directly address the requirements in issues #45126 and #45128 with no unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 2026-05-11-update-homepage-hero

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@website/assets/styles/pages/homepage.less`:
- Around line 1476-1478: The h1 font-size is inverted across breakpoints: the
base h1 rule (selector "h1") sets font-size: 24px, but a later media-breakpoint
rule that also targets "h1" increases it to 28px for narrower screens; update
the breakpoint rule so that h1 decreases (e.g., set the narrower-screen h1
font-size smaller than 24px) or refactor to a single responsive scale (adjust
the media query selector that currently overrides h1 to use a smaller value or
remove the override) to ensure headings get smaller on smaller devices.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ca1f9d9d-fc45-4cf8-ba6d-451a974ff050

📥 Commits

Reviewing files that changed from the base of the PR and between e942e06 and aa4ca99.

📒 Files selected for processing (4)
  • website/assets/styles/layout.less
  • website/assets/styles/mixins-and-variables/colors.less
  • website/assets/styles/pages/homepage.less
  • website/views/pages/homepage.ejs

Comment thread website/assets/styles/pages/homepage.less
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the marketing website homepage hero area and global header styling to align with updated wireframes/design system, including moving the testimonial quote out of the hero and adjusting masthead borders.

Changes:

  • Moved the homepage hero quote into a new “quote + statistics” section below the logo carousel.
  • Refined homepage hero layout and responsive styles (spacing, typography, background image sizing, hero height).
  • Replaced header box-shadow with a 1px bottom border using a new shared color token.

Reviewed changes

Copilot reviewed 1 out of 4 changed files in this pull request and generated 4 comments.

File Description
website/views/pages/homepage.ejs Restructures hero content and relocates quote + statistics into homepage content section.
website/assets/styles/pages/homepage.less Updates hero visuals, adds quote/stats layout styles, and adjusts responsive behavior across breakpoints.
website/assets/styles/mixins-and-variables/colors.less Adds @core-fleet-black-5 color variable for design-system consistency.
website/assets/styles/layout.less Updates masthead styling to use a bottom border instead of box-shadow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/assets/styles/pages/homepage.less
Comment thread website/assets/styles/pages/homepage.less Outdated
Comment thread website/assets/styles/pages/homepage.less
Comment thread website/assets/styles/pages/homepage.less Outdated
fleet-release
fleet-release previously approved these changes May 11, 2026
fleet-release
fleet-release previously approved these changes May 11, 2026
fleet-release
fleet-release previously approved these changes May 11, 2026
@eashaw eashaw merged commit 93e3cc4 into main May 11, 2026
9 checks passed
@eashaw eashaw deleted the 2026-05-11-update-homepage-hero branch May 11, 2026 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

~ga4-annotation This label is automatically applied to certain website changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Website request: Move hero quote on homepage Website request: Masthead style tweak

4 participants