Open
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request implements a comprehensive UI redesign focusing on modernizing the visual design system, updating layout patterns, and improving component consistency across the application. The changes introduce new typography components, update iconography to use Heroicons, expand the color palette for both light and dark modes, and transition pages to use a new AppContent wrapper with the default-no-header layout.
Key changes include:
- Introduction of new typography components (Title3, Headline, Footnote, Caption variants) following a design system
- Migration from grid-based layouts to flex-based layouts within a constrained content width (max-w-md)
- Addition of new color variables (v3) for neutral levels, primary green, secondary blue, and warning yellow shades
- EventCard component redesign with updated styling, new badge positions, and improved visual hierarchy
- Navigation menu updates with new icons and a special "Create" button with enhanced mobile styling
Reviewed changes
Copilot reviewed 49 out of 49 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
src/app/pages/session/view/index.vue |
Wraps content in AppContent and updates layout to default-no-header |
src/app/pages/session/edit/language.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/session/edit/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/session/edit/color-scheme.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/notification/index.vue |
Wraps content in AppContent, reorganizes script sections |
src/app/pages/guest/view/[id]/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/guest/unlock.vue |
Replaces custom max-width div with AppContent, updates layout logic |
src/app/pages/guest/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/event/view/[username]/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/event/view/[username]/[event_name]/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/event/view/[username]/[event_name]/guest.vue |
Refactors loader pattern to conditional rendering with AppContent |
src/app/pages/event/view/[username]/[event_name]/attendance.vue |
Refactors loader pattern to conditional rendering with AppContent |
src/app/pages/event/ingest/url.vue |
Wraps content in AppContent with section wrapper |
src/app/pages/event/ingest/image.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/event/index.vue |
Removes standalone title, wraps EventSearch in AppContent |
src/app/pages/event/edit/[username]/[event_name].vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/event/create.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/docs/browser-support.vue |
Wraps content in AppContent, reorganizes script sections |
src/app/pages/dashboard/index.vue |
Adds custom header with logo/buttons, updates loader components, changes layout |
src/app/pages/contact/index.vue |
Wraps content in AppContent, reorganizes script sections |
src/app/pages/attendance/index.vue |
Wraps content in AppContent and adds default-no-header layout |
src/app/pages/account/view/[username]/index.vue |
Adds new settings button to profile header |
src/app/pages/account/view/[username]/block/index.vue |
Replaces custom TopBar with LayoutPageTitle in AppContent |
src/app/pages/account/index.vue |
Removes standalone title, wraps AccountSearch in AppContent |
src/app/composables/head.ts |
Updates body class to use new color variables |
src/app/components/notification/NotificationIndicator.vue |
Updates to use new primary green color variable |
src/app/components/layout/menu/LayoutMenuSidebar.vue |
Updates styling to use new background color variable |
src/app/components/layout/menu/LayoutMenuItemList.vue |
Updates icons and adds Create button with special styling |
src/app/components/layout/menu/LayoutMenuItem.vue |
Refactors with v-slot pattern for icon attributes and updated typography |
src/app/components/layout/menu/LayoutMenuBottomNavigation.vue |
Updates styling to use new background color variable |
src/app/components/form/input/FormInputSearch.vue |
Redesigns with new styling and adds suffix slot |
src/app/components/event/card/EventCardTitle.vue |
Replaces conditional rendering with new typography components |
src/app/components/event/card/EventCard.vue |
Major redesign with new sizing, badge positioning, and color scheme |
src/app/components/event/EventSearch.vue |
Adds map toggle button with isMapVisible state |
src/app/components/app/typography/AppTypography*.vue |
Creates new typography components for design system consistency |
src/app/components/app/icon/AppIconHome.vue |
Switches to Heroicons HomeSolid icon |
src/app/components/app/icon/AppIconFavorite*.vue |
Switches to Heroicons Heart icons |
src/app/components/app/icon/AppIconCamera.vue |
Creates new camera icon component |
src/app/components/app/button/AppButtonNew.vue |
Creates new button component with primary/secondary variants |
src/app/components/app/button/AppButtonHeadless.vue |
Creates headless button component for composition |
src/app/components/app/AppContent.vue |
Creates new content wrapper with max-width constraint |
src/app/assets/css/vibetype.css |
Adds new v3 color variables for light and dark modes |
src/app/assets/css/app.css |
Adds subheadlineEmphasized utility class |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
10a90a3 to
9edc4b6
Compare
262bb60 to
36a460e
Compare
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.
📚 Description
This pull request introduces a significant UI refactor and style update across the app, focusing on modernizing the event card design, improving button and typography components, updating iconography, and expanding the color palette for both light and dark modes. The changes enhance consistency, accessibility, and maintainability of the codebase.
📝 Checklist