New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 800863 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

[css-grid] if there is an undefined element in the grid, other elements rows' height is changed strangely

Reported by asuka.o...@ft.com, Jan 10 2018

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36

Steps to reproduce the problem:
1. load the testcase.html with testcase.scss 
2. delete the line, 	 <div class="middle">middle</div> in testcase.html

What is the expected behavior?
It should not change the rows's height and rows span of bottom in grid-template-areas should still work even if some of the elements are undefined

What went wrong?
If there is an undefined element in the grid, other elements rows' height is changed strangely and rows span of bottom in grid-template-areas looks like doesn't work anymore.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 63.0.3239.132  Channel: stable
OS Version: OS X 10.12.6
Flash Version:
 
testcase.html
523 bytes View Download
testcase.scss
511 bytes Download
Screen Shot 2018-01-10 at 15.26.57.png
16.9 KB View Download
Screen Shot 2018-01-10 at 15.56.30.png
13.7 KB View Download

Comment 1 by asuka.o...@ft.com, Jan 10 2018

I forgot to add firefox in <Does this work in other browsers?>.
This issue doesn't happen in firefox(57.0.4).
Screen Shot 2018-01-10 at 18.41.40.png
37.5 KB View Download
Labels: Needs-Triage-M63
Tested the issue with the files provided in the Comment#0, we have downloaded both testcase.html & testcase.scss, kept both files in single folder and drag dropped testcase.html file into browser, but not able to see the behaviour as given in comment#0 screen shots. Even we have tried with the codepen to test this issue.

@Reporter:
Please find the attached screen cast for your reference and let us know if we missed anything in reproducing the issue, could you please provide any alternate test file/URL for further triaging the issue from TE end.

Thanks!
800863.mp4
2.0 MB View Download
Cc: viswatej...@techmahindra.com sc00335...@techmahindra.com
Components: Blink>CSS
Labels: Triaged-ET Needs-Feedback

Comment 5 by asuka.o...@ft.com, Jan 11 2018

I see the bug using sassmeister on Chrome/63.0.3239.132 and I have disabled all my chrome extensions as well when I check it.

open this url and delete <div class="middle">middle</div> line.
https://www.sassmeister.com/gist/4831a01cb16ecf019629004b084fd0a1

Hopefully you can see the bug.
Project Member

Comment 6 by sheriffbot@chromium.org, Jan 11 2018

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "sc00335628@techmahindra.com" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by asuka.o...@ft.com, Jan 11 2018

From the video, the html in codepen didn't have any <div class>. It looks different from what I have attached.

Comment 8 by r...@igalia.com, Jan 11 2018

Components: -Blink>CSS -Blink>Layout Blink>Layout>Grid
Status: WontFix (was: Unconfirmed)
@asuka.ochi next time is better if can upload an attached file to reproduce the issue or link to a codepen/jsbin/jsfiddle where the issue can be checked live.
I'm attaching the example BTW.

However I believe this is not a Chromium but a Firefox bug:
* https://bugzilla.mozilla.org/show_bug.cgi?id=1386921
* https://bugzilla.mozilla.org/show_bug.cgi?id=1386932

Check the following GitHub discussion for a detailed explanation:
https://github.com/rachelandrew/gridbugs/issues/2

bug-800863.html
1.1 KB View Download

Comment 9 by asuka.o...@ft.com, Jan 17 2018

Thank you for giving the information. I understood the reason why the css grid's height s are set like that, and also I am sorry I didn't attach testcase files following codepen/jsbin/jsfiddle's format. I won't do that next time!

Sign in to add a comment