nextui-org / nextui

🚀 Beautiful, fast and modern React UI library.
https://nextui.org
MIT License
21.93k stars 1.53k forks source link

enhancement: search Input width in navbar docs #3771

Open erikdemarco opened 1 month ago

erikdemarco commented 1 month ago

Is your feature request related to a problem? Please describe.

I like nextui, but it seems the developer just use this as a fun project and not serious about it.

For example when you see: https://nextui.org/docs/components/navbar#with-search-input

Then if you try modify the width, the search input looks so ugly when viewed in mobile width. It gives an impression that this example put there just to quickly finish the docs, dont care how it looks on mobile.

Describe the solution you'd like

please rewrite the "With Search Input" part in navbar docs

Describe alternatives you've considered

please rewrite the "With Search Input" part in navbar docs

Screenshots or Videos

Screenshot_2024-09-17_10-57-40

linear[bot] commented 1 month ago

ENG-1367 [Feature Request] Rewrite Search Input in navbar docs...

successbyte commented 1 month ago

Hi community, I am interested in this issue would you please assign it to me.

wingkwong commented 1 month ago

@successbyte assigned

wingkwong commented 1 month ago

@successbyte any update?

successbyte commented 1 month ago

Hi @wingkwong I am working on it please give me more time had a problem on running docs thanks

successbyte commented 1 month ago

Hi @wingkwong as the bellow picture all docs run good but the components are not working could you please guide me and I tried all contributing.md guidelines

next

wingkwong commented 1 month ago

looks fine on my side

image
successbyte commented 1 month ago

great I'm not gonna give up 😄 give me a little time trying on another laptop to se if it work

alexnguyennz commented 1 month ago

Might be related to line endings. I had the same issue, had to save the main mdx file e.g. navbar.mdx with LF instead of CLRF.

successbyte commented 1 month ago

Hey @alexnguyennz you saved my life It worked thanks man. now I'm solving the issue

successbyte commented 4 weeks ago

Hi @wingkwong finally I succeeded at running the project

anyway, do you want me to set min width to the input in small sizes?

and one thing thanks to gave me the chance (time)

successbyte commented 4 weeks ago

The changes I made to input I think it's good please confirm to make PR

For large screens:

Screenshot from 2024-10-10 22-20-37

For small screens:

Screenshot from 2024-10-10 22-21-41