Conversation
WalkthroughThe changes update the Marketing component to improve download link presentation, add an iOS App Store badge, and adjust layout for responsiveness. A new Android SVG icon component is introduced. The downloads page is enhanced to include separate, detailed cards for iOS and Android mobile apps, with appropriate icons, links, and status messages. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant MarketingComponent
participant DownloadsPage
User->>MarketingComponent: Views hero section
MarketingComponent-->>User: Shows Desktop and iOS download links
User->>DownloadsPage: Navigates to downloads page
DownloadsPage-->>User: Displays Desktop, iOS (with badge), and Android (coming soon) cards
Possibly related PRs
Poem
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. ✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Deploying maple with
|
| Latest commit: |
5bb4e7b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://42343d72.maple-ca8.pages.dev |
| Branch Preview URL: | https://mobile-app-links.maple-ca8.pages.dev |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
Added mobile app download options to the Maple application, including iOS App Store integration and Android placeholder for future release.
- Added new
/frontend/src/components/icons/Android.tsxcomponent following React SVG best practices - Enhanced
/frontend/src/routes/downloads.tsxwith dedicated mobile apps section featuring iOS App Store link - Updated Marketing component with responsive layout for desktop/mobile download options
- Implemented iOS platform detection for conditional rendering of download options
- Added App Store badge integration with proper linking to iOS app ID 6743764835
3 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
AnthonyRonning
force-pushed
the
mobile-app-links
branch
from
92c67bb to
5bb4e7b
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
frontend/src/routes/downloads.tsx (1)
135-177: Consider adding a subtitle to the mobile section for consistency.The desktop section has an informative subtitle that explains the value proposition, but the mobile section only has a title. Consider adding a subtitle for consistency and to provide additional context for users.
<div className="flex items-center gap-3 mb-6"> <Smartphone className="w-7 h-7" /> <span className="text-3xl font-light">Mobile Apps</span> + <span className="ml-3 text-xl text-[hsl(var(--marketing-text-muted))]"> + Access your private AI chat on the go + </span> </div>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
frontend/src/components/Marketing.tsx(1 hunks)frontend/src/components/icons/Android.tsx(1 hunks)frontend/src/routes/downloads.tsx(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- frontend/src/components/icons/Android.tsx
- frontend/src/components/Marketing.tsx
🧰 Additional context used
🧬 Code Graph Analysis (1)
frontend/src/routes/downloads.tsx (2)
frontend/src/components/icons/Apple.tsx (1)
Apple(3-16)frontend/src/components/icons/Android.tsx (1)
Android(3-16)
⏰ Context from checks skipped due to timeout of 90000ms (4)
- GitHub Check: build-ios
- GitHub Check: build-linux
- GitHub Check: build-macos (universal-apple-darwin)
- GitHub Check: Cloudflare Pages
🔇 Additional comments (3)
frontend/src/routes/downloads.tsx (3)
5-7: Icon imports appropriately added for mobile platforms.The addition of imports for smartphone, Apple, and Android icons properly prepare the file for the new mobile section.
134-140: Well-structured mobile section header.The section header for mobile apps follows the same design pattern as the desktop section, maintaining consistency across the page.
164-175: Android "coming soon" card implemented correctly.The Android card matches the iOS card's design while clearly indicating that the app is coming soon. This provides a good user experience by setting expectations.
| <div className="flex flex-col border border-[hsl(var(--marketing-card-border))] bg-[hsl(var(--marketing-card))]/75 text-foreground p-6 rounded-lg hover:border-foreground/30 transition-all duration-300"> | ||
| <div className="p-3 rounded-full bg-[hsl(var(--marketing-card))]/50 border border-[hsl(var(--purple))]/30 w-fit mb-4"> | ||
| <Apple className="w-6 h-6 text-[hsl(var(--purple))]" /> | ||
| </div> | ||
| <h3 className="text-xl font-medium mb-2">iOS</h3> | ||
| <p className="text-[hsl(var(--marketing-text-muted))] mb-6 flex-grow"> | ||
| Download our native iOS app for iPhones and iPads. | ||
| </p> | ||
| <div className="flex justify-center"> | ||
| <a | ||
| href="https://apps.apple.com/us/app/id6743764835" | ||
| className="inline-block" | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| > | ||
| <img | ||
| src="/app-store-badge.svg" | ||
| alt="Download on the App Store" | ||
| className="h-12" | ||
| /> | ||
| </a> | ||
| </div> | ||
| </div> |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
iOS App Store integration looks good.
The iOS card is well-implemented with appropriate styling, descriptive text, and a proper App Store badge link. The link opens in a new tab with appropriate security attributes.
🏁 Script executed:
#!/bin/bash
# Verify the App Store badge image exists
if [ -f "./public/app-store-badge.svg" ]; then
echo "✅ App Store badge SVG exists"
else
echo "❌ App Store badge SVG not found in public directory"
fiLength of output: 154
Add missing App Store badge asset
The App Store badge SVG referenced in frontend/src/routes/downloads.tsx is not present in the public directory, causing a broken image link for iOS downloads.
• Ensure public/app-store-badge.svg exists (or update the src path to point to the correct asset).
• Verify the SVG filename and location match the import in the <img> tag.
1 participant
Summary by CodeRabbit
New Features
Enhancements