Dev2 #123
Dev2 #123
Conversation
added app component and app link in footer.
✅ Deploy Preview for gobuildprod ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@copilot review it |
There was a problem hiding this comment.
Pull request overview
This pull request adds mobile app promotion functionality to the GoBuild website by introducing a new downloadable app component and integrating app download links throughout the site.
- Created a new
GoBuildDownloadAppcomponent featuring an interactive phone mockup with floating cards, testimonial, QR code, and Play Store badge - Integrated the app download component into the Contact Us and About Us pages
- Added a Play Store download badge to the footer for consistent visibility across the site
Reviewed changes
Copilot reviewed 4 out of 9 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
src/components/GoBuildDownloadApp.tsx |
New component showcasing the mobile app with animated elements, QR code, and download links |
src/pages/ContactUs.tsx |
Added GoBuildDownloadApp component to contact page |
src/pages/AboutUs.tsx |
Added GoBuildDownloadApp component to about page |
src/components/Footer.tsx |
Added Play Store badge with proper accessibility attributes |
public/AppScanner.jpg |
New QR code image asset for app downloads |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| {/* Invisible Button Overlay */} | ||
| <a | ||
| href="https://play.google.com/store/apps/details?id=com.go_build_app_version" |
There was a problem hiding this comment.
The Play Store URL is hardcoded in multiple places (lines 32, 114). Consider extracting this to a constant at the top of the file or in a configuration file to ensure consistency and easier maintenance.
| </div> | ||
| </div> | ||
| </section> */} | ||
| <GoBuildDownloadApp/> |
There was a problem hiding this comment.
Missing space before the opening tag. The formatting should be <GoBuildDownloadApp /> instead of <GoBuildDownloadApp/>.
| <GoBuildDownloadApp/> | |
| <GoBuildDownloadApp /> |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 9 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| </div> | ||
| {/* Play Store Badge */} | ||
| <a | ||
| href="https://play.google.com/store/apps/details?id=com.go_build_app_version" |
There was a problem hiding this comment.
The Play Store URL is hardcoded here and is the same URL used in GoBuildDownloadApp.tsx (lines 32 and 114). This creates code duplication across multiple files. Consider extracting this URL to a shared configuration file or constants file to maintain consistency and ease future updates.
src/pages/ContactUs.tsx
Outdated
| </div> | ||
| </div> | ||
| </section> | ||
| <GoBuildDownloadApp/> |
There was a problem hiding this comment.
[nitpick] Missing space after the opening tag. The component should have consistent spacing. Add a space after <GoBuildDownloadApp and before /> for consistency with JSX formatting standards.
| <GoBuildDownloadApp/> | |
| <GoBuildDownloadApp /> |
| <div className="mt-12 p-6 rounded-2xl backdrop-blur-xl bg-white/40 | ||
| border border-white/20 shadow-xl flex items-center justify-center gap-10"> |
There was a problem hiding this comment.
[nitpick] The div class string is split across two lines unnecessarily. Consider reformatting for better readability, either keeping it on one line or using a more consistent multi-line format.
| <div className="mt-12 p-6 rounded-2xl backdrop-blur-xl bg-white/40 | |
| border border-white/20 shadow-xl flex items-center justify-center gap-10"> | |
| <div className="mt-12 p-6 rounded-2xl backdrop-blur-xl bg-white/40 border border-white/20 shadow-xl flex items-center justify-center gap-10"> |
| <motion.a | ||
| href="https://play.google.com/store/apps/details?id=com.go_build_app_version" | ||
| whileHover={{ scale: 1.07 }} | ||
| className="transition-transform" | ||
| > | ||
| <img | ||
| src="/AppBadge(blue).png" | ||
| alt="Get it on Play Store" | ||
| className="w-40 rounded-xl shadow-lg" | ||
| /> | ||
| </motion.a> |
There was a problem hiding this comment.
The Play Store badge link lacks accessibility attributes. Add aria-label="Download GoBuild App on Google Play Store" to provide context for screen reader users and improve accessibility.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.