New issue
Advanced search Search tips

Issue 597952 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , All , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

div height:100% not rendered inside container with display:table

Reported by stefano....@gmail.com, Mar 25 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.49 Safari/537.36

Example URL:
http://codepen.io/sb_doc/pen/VaWLRr?editors=1100

Steps to reproduce the problem:
1. Open the submitted URL with Chrome Beta (50) or Canary (51)
2. Compare the output with other modern browsers (Chrome 49 too)

What is the expected behavior?
The two divs with text "HEIGHT 100%" should cover from "HEADER" (excluded) to "FOOTER" (excluded)

What went wrong?
The two divs with text "HEIGHT 100%" are tall as the text contained

Does it occur on multiple sites: N/A

Is it a problem with a plugin? No 

Did this work before? Yes Chrome 49 and before

Does this work in other browsers? Yes 

Chrome version: 50.0.2661.49  Channel: beta
OS Version: 10.0
Flash Version: Shockwave Flash 21.0 r0
 
firefox.png
7.0 KB View Download
chrome_beta_50.PNG
5.7 KB View Download
Labels: -Pri-2 -Type-Compat ReleaseBlock-Stable M-50 Needs-Bisect Pri-1 Type-Bug-Regression
Status: Untriaged (was: Unconfirmed)
Reproducible on Win 7/ 64 bit on the following builds:
Version 51.0.2690.0 canary (64-bit)
Version 50.0.2661.49 beta-m (64-bit)

This works fine on 49.0.2623.108 Stable build.

Marking this with RB label as this is a regression issue. Please feel free to change it.


Working on Bisect and i will provide the regression information soon.





Labels: OS-Mac
Owner: nyerramilli@chromium.org
Status: Assigned (was: Untriaged)
Unable to run bisect due to Bit 9 exception.
Issue seen on Mac OSX as well & this is reproducbile on Win 32 bit builds as well.

@Narayana, can you please get this bug bisected ?

Good Build --> 50.0.2626.0 [370259]
Bad Build --> 50.0.2629.0 [371142]
Cc: nyerramilli@chromium.org pucchakayala@chromium.org
Labels: -Needs-Bisect OS-Linux
Owner: dgro...@chromium.org
Bisect info:
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/e0b6a82cadecbda7e005921c2416fc7afb3542aa..1fee3b4957164505e70df4152fa31a2807fefa74

suspecting https://chromium.googlesource.com/chromium/src/+/8876584335b48c99cf8df552ef4d8efebb131041,
dgrogan@, Could you please check the above issue & help us in finding an owner it its not yours.

Good Build:50.0.2628.0
Bad Build: 50.0.2629.0

Able to reproduce the issue on Ubuntu 14.04 also.

Comment 4 by gov...@chromium.org, Mar 28 2016

A friendly reminder that M50 Stable is launching soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch by Apr-5. All changes MUST be merged into the release branch by 5pm on Apr-8 to make into the desktop Stable final build cut. Thanks!
Labels: -ReleaseBlock-Stable OS-All
Status: WontFix (was: Assigned)
This change is due to the % height work I started in issue 353580.

The short story is that you need to add height: 100% to a table-cell that surrounds your blocks: https://jsfiddle.net/8vd1pzbf/. I tested this in chrome 49, chrome 51, and FF 48. Let me know if this doesn't work for you and we'll iterate.

The longer story is that no two browsers render tables the same. The change I made in issue 353580 gets chrome closer to the css spec so we're going to keep it. The downside is that some sites inadvertently relied on the broken behavior. Your case is interesting because FF is broken in a way similar to chrome 49, which I hadn't seen before.
Labels: hasbisect

Sign in to add a comment