fix(web): allow dashboard header layout to wrap gracefully on smaller…#6804
Open
mayankdev-oss wants to merge 4 commits into
Open
fix(web): allow dashboard header layout to wrap gracefully on smaller…#6804mayankdev-oss wants to merge 4 commits into
mayankdev-oss wants to merge 4 commits into
Conversation
Contributor
|
Thanks, could you post a screenshot? |
Author
|
@qwerty287 I am currently experiencing some local environment issues preventing the full dashboard from rendering. I have verified the CSS fix by manually applying flex-wrap and gap classes to the header container in the browser inspector, which successfully resolves the overlap on narrower viewports. Once the CI build completes, if you can trigger a preview deployment, I will be happy to provide a live screenshot from there. |
Contributor
|
Our ci is down right now, we have to fix that first... |
Contributor
|
Works again, the pr image should be available |
Member
|
tested, cant se a behaviour difference in firefox |
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 fixes a layout bug on the repository dashboard where the action buttons overlap and clip into the search bar text field on narrower viewport sizes
By adding md:flex-wrap and switching to separate layout spacing variables md:gap-x-10 md:gap-y-2 on the main header row inside Header.vue the layout elements can now wrap into multiple rows safely if screen real estate runs tight. This keeps the UI completely accessible and looking great on smaller laptop or tablet screens while the team continues discussing long term button locations
Related Issue
Fixes #6739
Checklist