Add install page + Update download page#680
Add install page + Update download page#680tina-cloud-app[bot] wants to merge 21 commits intomainfrom
Conversation
Co-authored-by: Lewis Toh [SSW] <LewisToh@ssw.com.au>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR refactors the YakShaver installation page by renaming the route from /mcp to /install, adding a new "Install" button to the navigation bar, and introducing a new DownloadCards component to replace the TryItNow component. The redesigned page provides a cleaner interface for users to download the Desktop App or install the Teams App Extension.
Changes:
- Added new
DownloadCardscomponent for a more flexible installation options display - Renamed route from
/mcpto/installwith updated navigation - Added Windows SVG icon for the download buttons
- Replaced numbered steps with platform-based cards showing feature lists and download buttons
Reviewed changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tina/collectionSchema/pages.tsx | Registered DownloadCardsTemplate in the page blocks schema |
| public/svg/windows.svg | Added Windows logo SVG for the download button |
| content/pages/YakShaver/mcp.json | Removed old MCP page configuration |
| content/pages/YakShaver/install.json | Created new Install page with DownloadCards block and video display |
| content/navBars/YakShaver/YakShaver-NavigationBar.json | Added "Install" navigation item to the header |
| components/shared/blocksRenderer.tsx | Added DownloadCards case to the blocks renderer |
| components/shared/Blocks/DownloadCards.tsx | Implemented new DownloadCards component with card-based layout |
| components/shared/Blocks/DownloadCards.template.tsx | Created TinaCMS template definition for DownloadCards |
Comments suppressed due to low confidence (1)
components/shared/Blocks/DownloadCards.tsx:120
- The bottomLinks implementation is missing the
rel="noopener noreferrer"attribute when opening links in a new tab. This is a security concern as it can expose the page to potential tabnabbing attacks. The pattern is correctly implemented in the DownloadButton component (line 246), so the same should be applied here for consistency and security.
<NextLink
target="_blank"
{...rest}
className={cn(className, "hover:underline")}
href={link.url}
>
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Josh Berman [SSW] <137844305+joshbermanssw@users.noreply.github.com>
|
@Lewkans Great changes 👍 Small UI change: Please give the video thumbnail the same max-width as the Download Cards above. And ensure they use the same corner radius. For consistency.
|
✅ Done diff - I increased the download card widths instead, so that it takes up a bit more space and matches the width of the home page |


Before

After
