Website: update logos - #49695
Conversation
There was a problem hiding this comment.
Warning
- Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.
Pull request overview
Updates the website’s customer-logo displays (solutions pages and the logo carousel) to replace older logos with new customer logos.
Changes:
- Swap “Mozilla” logo slots on multiple solutions pages for Flywire, Mollie, Faire, and Rivian.
- Update the logo carousel component and related solution-page styles (diff content excluded by policy).
Reviewed changes
Copilot reviewed 5 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| website/views/pages/visibility-and-reporting.ejs | Replace Mozilla logo entry with Flywire in both desktop/mobile logo grids. |
| website/views/pages/software-management.ejs | Replace Mozilla logo entry with Flywire in both desktop/mobile logo grids. |
| website/views/pages/security-and-control.ejs | Replace Mozilla logo entry with Rivian (but class name appears incorrect; see comments). |
| website/views/pages/linux-management.ejs | Replace Mozilla logo entry with Mollie in both desktop/mobile logo grids. |
| website/views/pages/device-management.ejs | Replace Mozilla logo entry with Faire in both desktop/mobile logo grids. |
| website/assets/styles/pages/linux-management.less | (Content excluded) Styles update for Linux management page logos. |
| website/assets/styles/pages/device-management.less | (Content excluded) Styles update for device management page logos. |
| website/assets/styles/pages/software-management.less | (Content excluded) Styles update for software management page logos. |
| website/assets/styles/pages/security-and-control.less | (Content excluded) Styles update for security and control page logos. |
| website/assets/styles/pages/visibility-and-reporting.less | (Content excluded) Styles update for visibility and reporting page logos. |
| website/assets/js/components/logo-carousel.component.js | (Content excluded) Logo carousel component update. |
Files excluded by content exclusion policy (6)
- website/assets/js/components/logo-carousel.component.js
- website/assets/styles/pages/device-management.less
- website/assets/styles/pages/linux-management.less
- website/assets/styles/pages/security-and-control.less
- website/assets/styles/pages/software-management.less
- website/assets/styles/pages/visibility-and-reporting.less
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughUpdated page logo grids to replace Mozilla with Faire, Mollie, Rivian, and Flywire in desktop and mobile layouts. Removed obsolete Mozilla-specific responsive styling and adjusted Grafana logo sizing on the security-and-control page. Removed Mozilla from both Group one sections of the logo carousel and changed their count from eight to seven. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Warning
- Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.
Pull request overview
Copilot reviewed 5 out of 14 changed files in this pull request and generated 1 comment.
Files excluded by content exclusion policy (6)
- website/assets/js/components/logo-carousel.component.js
- website/assets/styles/pages/device-management.less
- website/assets/styles/pages/linux-management.less
- website/assets/styles/pages/security-and-control.less
- website/assets/styles/pages/software-management.less
- website/assets/styles/pages/visibility-and-reporting.less
Comments suppressed due to low confidence (1)
website/views/pages/security-and-control.ejs:221
- This logo entry is missing a per-logo CSS class, unlike the other items in the same grid (e.g., fastly/easygo/calendly). If the page styling relies on these classes for sizing/spacing, Rivian will render inconsistently. Add a class (e.g., "rivian") to match the existing pattern.
<div purpose="logo">
<img alt="Rivian logo" src="/images/logos/logo-rivian-color-200x40@2x.png">
</div>
Closes: https://github.com/fleetdm/confidential/issues/16891
Changes:
Summary by CodeRabbit