feat: add Desktop to homepage feature card platforms#1042
Conversation
✅ Deploy Preview for lynx-doc ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ 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.
Pull request overview
This PR updates the homepage “Write Once, Render Anywhere” feature card to add a new Desktop action row, showing macOS + Windows icons and linking into the existing “Integrate with Existing Apps” guide with the macOS tab preselected.
Changes:
- Add a new “Desktop” action row (macOS + Windows icons) to the homepage feature card actions.
- Introduce
IconMacOS/IconWindowsReact SVG components and wire them into the homepage feature config. - Add CSS module styles for the new icons and add the
home-icon-macos.svgasset under the home assets directory.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/components/home-comps/features/index.tsx | Adds the new “Desktop” action row (macOS + Windows icons) and link. |
| src/components/home-comps/features/index.module.less | Adds CSS module styling for the new macOS/Windows icons. |
| src/components/home-comps/features/icon.tsx | Adds IconMacOS/IconWindows exports and adjusts icon imports. |
| docs/public/assets/home/home-icon-macos.svg | Adds the macOS SVG used by the new Desktop row. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .macos-icon { | ||
| width: 18px; | ||
| height: 18px; | ||
| path { | ||
| fill: var(--home-blog-btn-color) !important; |
| const IconMacOS = () => { | ||
| return <MacOSIcon className={styles['macos-icon']} />; | ||
| }; | ||
|
|
||
| const IconWindows = () => { | ||
| return <WindowsIcon className={styles['windows-icon']} />; | ||
| }; |
Adds a Desktop action button with macOS + Windows logos to the "Write Once, Render Anywhere" feature card, linking to the desktop integration guide. Change-Id: I0980dd4fc98abd1a7bf6a923856fb80284ec200b
ad13b53 to
f1af412
Compare
Summary
<APISummary />(viahome-icon-macos.svgcopied from the compat-table icon set) so the homepage stays consistent with the API docsguide/start/integrate-with-existing-apps.html?platform=macosThe desc copy already mentions "mobile and desktop", so no copy change was needed.
Test plan
/and/zh/— confirm the Desktop row renders below Web with both macOS and Windows icons--home-blog-btn-color