search

FormidableLabs / nextjs-sanity-fe

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

Mobile bugs #150

Closed nesarkhadem closed 1 year ago

nesarkhadem commented 2 years ago

  1. Top hamburger nav is fixed to the top the website, https://www.loom.com/share/f5d4d526df8f4ef5b1c934cd438c330e

  2. sorting and filtering is at the bottom of the page, this possibly needs UI enhancements

Image

  1. there seems a scrolling bar in minicart, which has no implications

Image

  1. price sorting is working weirdly https://www.loom.com/share/c80ca243eb0e442aa7bed25fd4e34eb5

  2. Alphabetical sorting also facing the same issue

gksander commented 2 years ago

Addressed via #151, except 2. With 2, I figured beneath the products was a more appropriate place for the sorting/filtering (since it's kind of a long list). Could definitely be improved, but I figured for an MVP, stuffing it at the bottom would be alright.

elizamander commented 2 years ago

@nesarkhadem This is ready for QA again

nesarkhadem commented 2 years ago

Completed and all concerned issues are fixed

sarmeyer commented 1 year ago

Upon checking, only 2 is still an "issue". I'm going to create a new task for it as an enhancement

sarmeyer commented 1 year ago

https://github.com/FormidableLabs/nextjs-sanity-fe/issues/170