Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Search bar max width navbar #7837

Closed
exezbcz opened this issue Oct 26, 2023 · 2 comments · Fixed by #7842
Closed

Search bar max width navbar #7837

exezbcz opened this issue Oct 26, 2023 · 2 comments · Fixed by #7842
Assignees
@prachi00
Labels
A-navbar related to navbar components A-search related to search components p3 non-core, affecting less than 40% visual change not affecting ux of app

Comments

@exezbcz
Copy link
Member

exezbcz commented Oct 26, 2023

limit width of searchbar in navbar

  • its too big, takes too much attention
  • on desktops it does not look good
  • when scrolling - the cute search bar changes to the monster in navbar

image
image

  • change the max width to 600px
  • put equal space between the logo/searchbar/right part
@exezbcz exezbcz added p3 non-core, affecting less than 40% A-search related to search components A-navbar related to navbar components visual change not affecting ux of app labels Oct 26, 2023
@prachi00
Copy link
Member

👋

@kodabot
Copy link
Collaborator

kodabot commented Oct 26, 2023

ASSIGNED - @prachi00 🔒 LOCKED -> Saturday, October 28th 2023, 10:41:33 UTC -> 36 hours

prachi00 added a commit to prachi00/nft-gallery that referenced this issue Oct 27, 2023
@prachi00
kodadot#7837 search bar updates
c24908a
@prachi00 prachi00 mentioned this issue Oct 27, 2023
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prachi00 prachi00

Labels
A-navbar related to navbar components A-search related to search components p3 non-core, affecting less than 40% visual change not affecting ux of app
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#7837 search bar updates prachi00/nft-gallery
3 participants
@prachi00 @exezbcz @kodabot