feat(frontend): refresh landing and scan experience - #10
Merged
KazanderDad merged 3 commits intoNov 9, 2025
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR implements a comprehensive UI redesign for the Peer Mapper application, transforming it from a basic list-style interface into a polished, modern design with a cohesive dark theme and improved user experience.
- Introduces new
AppHeaderandAppFootercomponents with navigation and branding - Refactors the home page to conditionally render marketing content for unauthenticated users and quick actions for authenticated users
- Splits the monolithic
/scanpage into separate/scan/my-qrand/scan/cameraroutes for better UX
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/components/AppHeader.tsx | New header with navigation, user profile display, and avatar initials generation |
| frontend/src/components/AppFooter.tsx | New footer with developer shortcuts and branding |
| frontend/src/app/page.tsx | Refactored to show marketing content when unauthenticated, quick actions when signed in |
| frontend/src/app/layout.tsx | Updated to use new header/footer and removed old UserSessionSummary |
| frontend/src/app/globals.css | Updated CSS variables for dark theme design system |
| frontend/src/app/(routes)/scan/page.tsx | Simplified to redirect to /scan/my-qr |
| frontend/src/app/(routes)/scan/my-qr/page.tsx | New page for displaying user's QR code |
| frontend/src/app/(routes)/scan/camera/page.tsx | Moved scan/verify logic from old /scan page |
| frontend/package.json | Updated lint command to use ESLint directly |
| frontend/eslint.config.mjs | Added rule overrides for React hooks |
| frontend/tests/scan-camera-page.test.tsx | Updated test to reference new page location and wrap store resets in act |
| frontend/tests/navigation.test.tsx | Removed obsolete test for old home page link structure |
| frontend/tests/home-page.test.tsx | New tests for updated home page conditional rendering |
| frontend/tests/app-header.test.tsx | New tests for header component |
💡 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Testing
Codex Task