FormidableLabs / nextjs-sanity-fe

NextJS Demo site with Sanity CMS
https://nextjs-sanity.formidable.dev/
29 stars 6 forks source link

170 Possible filter UI improvement #186

Closed luisabarca closed 1 year ago

luisabarca commented 1 year ago

z-index bug

Screenshot 2023-02-01 at 23 44 12

Video

https://user-images.githubusercontent.com/186492/216249206-e438abcd-94fd-4c9f-ae01-67faa0fc0e85.mov

Before

nextjs-sanity formidable dev_products

After

Screenshot 2023-02-02 at 0 27 03 Screenshot 2023-02-01 at 23 34 41
vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated
nextjs-sanity-fe ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 6, 2023 at 10:34PM (UTC)
luisabarca commented 1 year ago

This is great! I left a couple comments. Also, WDYT about adding a filter count to the filters button on smaller screen sizes? ex. Filters (1)

Sounds good, maybe we'll need another ticket, because we'll need to change the behavior of the checkboxes