feat: homepage hero, 50/50#8
Merged
Merged
Conversation
✅ Deploy Preview for bs-said-yes ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
ben-basten
changed the title
There was a problem hiding this comment.
Pull request overview
This PR introduces two new content modules (Homepage Hero and Fifty/Fifty), enhances the footer with styling, and adds support for a 12-column grid system. The changes enable more flexible page layouts and improve the visual presentation of the homepage.
Changes:
- Added two new GraphQL fragments and query integration for Homepage Hero and Fifty/Fifty modules
- Created new icon components (IconCalendar, IconPinHeart) for the Homepage Hero
- Added 12-column grid utility and typography utilities for eyebrow text and copy width constraints
- Updated module renderer to support the new modules with cleaner import naming
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| server/graphql/queries/page-standard.query.ts | Added imports and fragment references for FiftyFifty and Link fragments |
| server/graphql/modules/homepage-hero.fragment.ts | Updated to query for eyebrow, date, location, and multiple images instead of copy and single image |
| server/graphql/modules/fifty-fifty.fragment.ts | New fragment definition for the Fifty/Fifty module with heading, copy, CTAs, and image |
| app/components/svg/IconPinHeart.vue | New heart-shaped location pin icon component |
| app/components/svg/IconCalendar.vue | New calendar icon component |
| app/components/render/ModuleRenderer.vue | Updated imports to use cleaner naming and registered FiftyFifty module |
| app/components/navigation/Footer.vue | Added divider and spacing utilities for improved visual styling |
| app/components/modules/HomepageHero.vue | Complete redesign with grid layout, eyebrow text, date/location display, and multiple image grid |
| app/components/modules/FiftyFifty.vue | New 50/50 split layout module with content on one side and image on the other with configurable alignment |
| app/components/base/ContentContainer.vue | Added optional grid prop to support 12-column grid layout |
| app/components/PageStandard.vue | Removed redundant h1 heading that was replaced by module-specific headings |
| app/assets/css/main.css | Added grid-12 utility for 12-column grid layout |
| app/assets/css/_typography.css | Added eyebrow text and max-width copy utilities |
| app/assets/css/_component.css | Added divider utility for visual separators |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Uh oh!
There was an error while loading. Please reload this page.