feat: Move Teams banner to all pages with dismissible functionality #368
Conversation
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
![roomote[bot]](https://avatars.githubusercontent.com/in/1546624?s=80&v=4){kind=small}
- Created new TeamsBanner component with localStorage-based dismiss - Added banner above navbar in main Layout component - Removed homepage-only banner from DocBreadcrumbs - Banner now appears on all pages and can be dismissed - Dismiss state persists across sessions via localStorage
roomote
bot
requested review from
cte,
hannesrudolph and
mrubens
as code owners
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Banner now appears above navbar instead of covering it - Removed fixed positioning and body class management - Banner flows naturally in the document layout
- Keep flex-direction as row on mobile - Reduce font size for mobile devices - Add white-space nowrap to prevent text wrapping - Adjust padding for better mobile appearance
Removed border and box-shadow from TeamsBanner styles.
mrubens
approved these changes
Sep 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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 implements the requested changes to move the Teams banner from homepage-only to all pages, positioned above the navbar with dismissible functionality stored in localStorage.
Changes Made
TeamsBannercomponent with localStorage-based dismiss functionalityLayoutcomponent so it appears on all pagesImplementation Details
Testing
Resolves the Slack request: "Can you make it so the teams banner on the homepage gets put on all pages instead, above the navbar? It should be dismissible with that stored in local storage."
Important
Introduces a dismissible
TeamsBanneron all pages, replacing the old homepage-only banner, with localStorage persistence and responsive styling.TeamsBannercomponent added with dismiss functionality using localStorage inindex.tsx.Layoutinindex.js.DocBreadcrumbsinindex.tsx.TeamsBannerinstyles.module.csswith responsive design for mobile.TeamsBannerstores preference in localStorage.This description was created by
for 1d52082. You can customize this summary. It will automatically update as commits are pushed.