Unable to confirm an adress on www.foodora.cz due to button being outside the viewport and parent container unscrollable
Categories
(Web Compatibility :: Site Reports, defect, P1)
Tracking
(Webcompat Priority:P1, Webcompat Score:8)
People
(Reporter: ksenia, Unassigned)
References
()
Details
(Keywords: webcompat:platform-bug, webcompat:site-report, webcompat:sitepatch-applied)
User Story
platform:android impact:workflow-broken affects:all configuration:general branch:release user-impact-score:800
Attachments
(1 file)
|
1.28 KB,
text/html
|
Details |
We've got a report in https://github.com/webcompat/web-bugs/issues/130975 where a user is unable to pick an address and make an order since the button to confirm the address is outside of the viewport and it's impossible to scroll to it.
To reproduce:
- Visit https://www.foodora.cz/en/restaurant/a1wt/indicka-restaurace-everest in Firefox Nightly (123.0a1) in RDM or on the actual device.
- Click on the plus sign to add a meal, wait for "What's your address?" to appear and click on "Add an address".
- Move the map to a nearest building to make sure it picks up the address.
Expected:
Can scroll to "Find vendors" button underneath the map.
Actual:
The button parent container is unscrollable, can't reach the button.
| Reporter | ||
Comment 1•2 years ago
|
||
I've attached a slightly reduced testcase (reproducible on RDM as well).
| Reporter | ||
Comment 2•2 years ago
|
||
Thinking this could be related to bug1663634
Comment 3•2 years ago
|
||
Not really, this looks like grid layout not sizing the scroll container (.content) correctly here. Daniel, Ting-Yu, you've been looking at grid issues recently, and flexbox has similar rules IIRC. Any chance you can take a look?
Comment 4•2 years ago
|
||
It seems max-height: 100% on the grid container .wrapper made the grid item .content unable to scroll. If I change max-height: 100% to height: 100%, it works as expected.
Flex container works well on this testcase. I test it via adding display: flex; flex-direction: column; to .wrapper, and I can scroll the .content flex item.
Updated•2 years ago
|
Comment 5•2 years ago
|
||
The issue is roughly this:
- the grid's height is indefinite (content-based), though its constrained by
max-height:100% - that means its grid-row (sized to fill the grid with 1fr) and grid item (sized to fill the containing block with
height:100%) are also effectively indefinitely-sized and size-to-content. - So we don't end up resolving the
height:100%on the grid item to actually be the (max-height-constrained) size of the parent grid.
I think this is essentially a form of bug 1481876.
(I also don't think it's actually related to the viewport-compat bug, even though the results are similar.)
| Reporter | ||
Updated•2 years ago
|
Comment 6•2 years ago
|
||
We know what the underlying issue is here, so let's try to build a site patch.
Updated•2 years ago
|
| Reporter | ||
Updated•2 years ago
|
| Reporter | ||
Updated•2 years ago
|
| Reporter | ||
Comment 7•2 years ago
|
||
A sitepatch is shipped in bug1876363
| Reporter | ||
Updated•2 years ago
|
| Reporter | ||
Updated•2 years ago
|
Updated•1 year ago
|
Comment 8•1 year ago
|
||
I forgot to close this bug out after we removed the site-patch in bug 1890350. Doing so now.
Description
•