Fix Navbar Overlap and Enhance Discord Invite UI on Support Page - #8182
Fix Navbar Overlap and Enhance Discord Invite UI on Support Page#8182rythmlongia wants to merge 1 commit into
Conversation
|
This PR resolves the navbar overlap issue and improves the Discord invite UI, which is crucial for user navigation and engagement. It's worth noting that there have been previous discussions regarding the Discord invite link to ensure consistency across the application. Please refer to the related decision on updating the support page to use the canonical Discord invite link for context. Thank you for your contributions! |
📦 Next.js Bundle Size Report (Gzipped Sizes)
📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Thanks for fixing the navbar overlap and enhancing the Discord button UI!
I noticed your commit message is \Fix support page navbar UI, which doesn't follow the Conventional Commits format required in our checklist. Could you please amend your commit message to something like \ix(support): fix navbar overlap and enhance discord invite ui\ and force push?
I'll approve this right after that's fixed! 🎨
|
@rythmlongia is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Aamod007 |
Aamod007
left a comment
There was a problem hiding this comment.
Thanks for looking into this styling issue on the support page!
However, completely removing the \ixed\ class from the <nav>\ is not the best approach here, as it breaks the sticky behavior of the navbar when the user scrolls down the page. The standard way to fix content overlapping a \ixed\ navbar is to add adequate padding (like \pt-20\ or similar) to the top of the main content container (<main>\ or the next <div>) so it sits below the navbar.
Additionally, using \mr-10\ on the Discord button feels a bit like a workaround. Consider using flexbox gap properties on the parent container if you're trying to separate it from adjacent elements.
Please restore the \ixed\ class and apply top padding to the main content instead.
Label Justification:
- \level:beginner: CSS padding/margin and positioning fixes fall under beginner-level styling tasks.
- \ ype:bug: Fixes layout overlap.
- \ ype:design: Modifies UI layout and spacing.
- \mentor:Aamod007: Assigned as required.
|
@Aamod007 I understood the issue and have already corrected the Discord button by removing the margin-based spacing.
|
|
If you are still working on this, please push your latest changes or leave a comment to keep it active. |


Description
Fixes # 8104
Fixes #8104