Shopify / dawn

Shopify's first source available reference theme, with Online Store 2.0 features and performance built-in.
Other
2.48k stars 3.32k forks source link

Quick add modal display issues when selecting variant on mobile #2098

Closed sofiamatulis closed 1 year ago

sofiamatulis commented 1 year ago

Describe the current behavior A store has the media desktop layout on the product page set to thumbnail carousel, there is a long list of variants, and the quick add button on the featured collection section is enabled. When viewing the Quick Add modal on a mobile device, if the user scrolls down and selects a variant, the content moves up out of view and the bottom gets pushed up. At this point the user can't scroll and use the product form correctly.

Describe the expected behavior There should be no empty white space added to it and the user should be able to scroll

Steps to replicate (On mobile)

Additional context/screenshots Video: https://screenshot.click/04-04-1e8xk-i13j8.mp4

andrewetchen commented 1 year ago

I was looking at this issue today. I was able to reproduce the issue as described on Dawn 7.0.1

https://user-images.githubusercontent.com/28404165/208780924-dcb0eeb4-9834-4fd7-b5b1-b46aa476178b.mp4

However, I was unable to reproduce on the latest main (as of December 20 2022). I'm assuming this issue got fixed in https://github.com/Shopify/dawn/pull/2031 and potentially other merged PRs

https://user-images.githubusercontent.com/28404165/208781396-fb9e752e-8de6-4094-841d-10d405f6e929.mp4

Notes on replication steps

You can follow your replication steps above but I found that all I needed to do was:

andrewetchen commented 1 year ago

This has already been fixed, see: https://github.com/Shopify/dawn-private/issues/77