1229327 - [css-grid] Grid container should report overflow so that it can be scrolled

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Javier Fernandez]

Attachment: bug-sizing-with-scroll.html

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Iceweasel/38.2.1
Build ID: 20150828090403

Steps to reproduce:

Loading the attached test case.


Actual results:

The grid container it's sized to fit just the scrollbar size, ignoring the size of the grid, which is 100px x 100px.


Expected results:

The grid container's size should have been adapted to the grid's size, which is 100px x 100px, using the scroll bar to deal with the item's  overflowing area.

Comment 1

[Mats Palmgren (inactive)]

I mostly fixed this in bug 1227162 a couple of days ago.

We still don't propagate any overflow areas though so you can't scroll it
at the moment.

Comment 2

[Mats Palmgren (inactive)]

Actually we do propagate child overflow areas:
http://hg.mozilla.org/mozilla-central/annotate/e7b2e8a061f9/layout/generic/nsGridContainerFrame.cpp#l2904

So this is working just fine now.  I was confused because we don't have scrollbar thumbs
in Nightly at the moment so it's not possible to detect any scroll progress in the attached
testcase since it's just a uniform green item with no text.  Adding a bit of text and using
mouse-wheel scroll shows that it's working fine.

So this was fixed by bug 1227162.