osmapp icon indicating copy to clipboard operation
osmapp copied to clipboard

Updated ui.

Open orl0pl opened this issue 1 year ago • 3 comments

I made feature panel and search bar same size, also I made search bar more round.

orl0pl avatar May 05 '23 15:05 orl0pl

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
osmapp ✅ Ready (Inspect) Visit Preview 1 resolved May 6, 2023 2:25pm

vercel[bot] avatar May 05 '23 15:05 vercel[bot]

Hi, thanks for your PR. The ui looks like a move in a good direction, at least on desktop (similar PR here). But can you please shortly describe your motivation? Does it solve some issue? Or did you just perhaps wanted to make osmapp more beatiful? :)

I found some issues:

  1. the panel closing button on homepage is out of the searchbox.
  2. the preview closing button on mobile as well
  3. the padding of the searchbox is different then padding of map controls on the right top corner
  4. i think the left bottom corner is missing rounding.
  5. the scrollbar stopped disappearing
  6. I dont like this version on mobile, i would say the previous one was more useful - providing more space in the limited setup. I am not sure what to do there yet. Using the "signature" red searchbox only on mobile doesnt feel like a solution. image

Overall I would need some time to decide if this is the way to move osmapp. Especially with limited time i cant invest into style revamp while there are more pressing issue like #131.

zbycz avatar May 06 '23 05:05 zbycz

Closing in reference to Tailwind/Headless change

Flohhhhh avatar Jun 14 '23 20:06 Flohhhhh