fix(marketing): platform-aware download CTA icons and cursor icon#2702
fix(marketing): platform-aware download CTA icons and cursor icon#2702meetpateltech wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
ApprovabilityVerdict: Approved This PR contains only client-side UI changes to the marketing download CTAs - updating icon display and link text based on detected OS. The changes are cosmetic, self-contained, and have no backend or security implications. You can customize Macroscope's approvability policy. Learn more. |
What Changed
:global()Why
The download CTA was showing multiple platform icons at once and the Linux Tux icon was not rendering correctly due to Astro scoped CSS behavior. This update improves platform-specific CTA clarity, fixes icon visibility issues, and updates the cursor icon to a lighter variant for better visibility on dark backgrounds.
Fixes #2683
UI Changes
1. Hero Section:
2. Linux Icon:
3. Cursor Icon:
4. Final CTA Section:
Checklist
Note
Low Risk
Low risk marketing-page UI changes; main risk is unintended DOM/CSS behavior that could hide/remove the wrong icon or mislabel the alternate-platform link.
Overview
Makes the marketing homepage download CTAs platform-aware: the JS now updates the secondary CTA to list only other platforms, and removes non-matching
.dl-iconelements so only the current OS icon remains.Fixes Astro CSS scoping for download icons by switching icon rules to
:global()and tightening default/data-platformdisplay logic.Updates
cursor_light.svgto a simplified light-filled variant for better visibility on dark backgrounds.Reviewed by Cursor Bugbot for commit a16a2ce. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix platform-aware download CTA icons and 'other platforms' link text on marketing index
.dl-iconelements from the DOM after platform detection.html[data-platform]CSS selectors as a fallback.cursor_light.svgasset is added to the marketing public harnesses.Macroscope summarized a16a2ce.