Fix header/logo layout and load perfume sizes from Sanity#4
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (19)
📝 WalkthroughWalkthroughThis PR refreshes marketing copy and layout across About, Home, Hero, BrandStory, and Shop pages; adds a ChangesMarketing content and layout updates
Product sizes data flow
Developer page CTA extraction and Button link attributes
Estimated code review effort: 2 (Simple) | ~15 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant DeveloperCtas
participant Browser
User->>DeveloperCtas: click Email or WhatsApp button
DeveloperCtas->>Browser: window.open(mailto: or wa.me URL, "_blank", "noopener,noreferrer")
Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the storefront UI (header/footer/home/about/shop) and product data fetching so that perfume size variants are sourced from Sanity and surfaced throughout the app, while also tightening logo/header presentation and adjusting homepage layout.
Changes:
- Fetch and propagate
sizesfrom Sanity product documents (queries + page/component mappings). - Update header/footer branding to use the logo image and adjust homepage layout spacing.
- Add a new “WhyChooseUs” homepage section and refresh marketing copy across multiple pages.
Reviewed changes
Copilot reviewed 18 out of 21 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| types/components.types.ts | Extends ButtonProps to support target/rel for link-style buttons. |
| sanity/lib/client.ts | Adds sizes[] selection to product GROQ queries (including featured). |
| lib/constants.ts | Notes removal of local perfume-size fallback approach in favor of Sanity-defined sizes. |
| components/ui/Button.tsx | Passes target/rel through when rendering a Link. |
| components/layout/Navbar.tsx | Replaces text logo with next/image logo and simplifies navbar imports/structure. |
| components/layout/Footer.tsx | Replaces text logo with next/image logo in the footer. |
| components/layout/ClientLayout.tsx | Tweaks studio wrapper styling and adds a homepage-only spacer after the navbar. |
| components/home/WhyChooseUs.tsx | New homepage “Why Choose Us” feature grid section. |
| components/home/Hero.tsx | Updates hero headline, tagline, copy, and stats. |
| components/home/FeaturedProducts.tsx | Ensures featured product mapping includes sizes. |
| components/home/BrandStory.tsx | Refreshes brand story copy and minor formatting tweaks. |
| app/shop/ShopClient.tsx | Updates shop page header title/subtitle copy. |
| app/shop/page.tsx | Ensures shop product mapping includes sizes. |
| app/product/[slug]/page.tsx | Ensures product page mapping includes sizes. |
| app/page.tsx | Adds WhyChooseUs to the homepage composition. |
| app/developer/page.tsx | Refactors developer page (local image, CTA extraction) and adjusts background element sizing. |
| app/developer/DeveloperCtas.tsx | New client component for developer CTAs (Contact/WhatsApp). |
| app/about/page.tsx | Significant about-page content/layout refresh, adds staggered animation-delay classes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div className="pt-32 pb-20 overflow-hidden relative selection:bg-gold/30"> | ||
| {/* Background Decorative Elements */} | ||
| <div className="absolute top-0 left-1/2 -translate-x-1/2 w-full h-[600px] bg-gold/3 blur-[120px] rounded-full pointer-events-none" /> | ||
| <div className="absolute top-0 left-1/2 -translate-x-1/2 w-full h-150 bg-gold/3 blur-[120px] rounded-full pointer-events-none" /> |
Comment on lines
+9
to
+21
| <Button | ||
| onClick={() => | ||
| window.open( | ||
| "mailto:azeemkhandsari@gmail.com", | ||
| "_blank", | ||
| "noopener,noreferrer", | ||
| ) | ||
| } | ||
| variant="primary" | ||
| className="gap-2" | ||
| > | ||
| <Mail size={16} /> Contact | ||
| </Button> |
Comment on lines
+22
to
+34
| <Button | ||
| onClick={() => | ||
| window.open( | ||
| "https://wa.me/917895411144", | ||
| "_blank", | ||
| "noopener,noreferrer", | ||
| ) | ||
| } | ||
| variant="outline" | ||
| className="gap-2" | ||
| > | ||
| <MessageCircle size={16} /> WhatsApp | ||
| </Button> |
Comment on lines
+42
to
+49
| <Image | ||
| src="/MS_Signature_Logo.jpg" | ||
| alt="MS Signature Logo" | ||
| width={120} | ||
| height={120} | ||
| className="w-auto h-12 sm:h-14 md:h-16 object-contain" | ||
| priority | ||
| /> |
Comment on lines
+56
to
+57
| <Link href="/" className="flex items-center gap-4 mb-6 group" aria-label="Home"> | ||
| <Image src="/MS_Signature_Logo.jpg" alt="MS Signature Logo" width={140} height={40} className="object-contain" /> |
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.
This PR tightens the header/logo presentation, removes the extra homepage spacing issue from other pages, and ensures perfume size variants are sourced from Sanity instead of relying on local fallbacks.
Summary by CodeRabbit
New Features
Enhancements