RESOLVED FIXED 85581
Percentage height replaced elements sometimes cause overflow of table contents
https://bugs.webkit.org/show_bug.cgi?id=85581
Summary Percentage height replaced elements sometimes cause overflow of table contents
Terry Anderson
Reported 2012-05-03 22:32:11 PDT
Reported here: http://code.google.com/p/chromium/issues/detail?id=122806 To view an example of the bug, in chromium visit http://support.google.com/ics/nexus/bin/answer.py?hl=en&answer=1637532 and you will notice that the Settings icons are much bigger than intended. The image has a percentage height of 60%. From inspecting the page, it appears that this is relative to the <table> rather than to its parent <p>, but the CSS specs say that the containing block of this image should in fact be the <p> element.
Attachments
Patch (1.89 KB, patch)
2012-05-03 23:06 PDT, Terry Anderson
no flags
Patch (64.08 KB, patch)
2012-05-11 13:45 PDT, Terry Anderson
no flags
Patch (5.56 KB, patch)
2012-05-18 12:04 PDT, Terry Anderson
no flags
Terry Anderson
Comment 1 2012-05-03 23:06:02 PDT
WebKit Review Bot
Comment 2 2012-05-03 23:09:25 PDT
Attachment 140163 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Eric Seidel (no email)
Comment 3 2012-05-03 23:09:45 PDT
Comment on attachment 140163 [details] Patch This needs tests. It's also certainly not chromium only.
Terry Anderson
Comment 4 2012-05-11 13:45:25 PDT
Terry Anderson
Comment 5 2012-05-11 13:55:56 PDT
(In reply to comment #4) > Created an attachment (id=141489) [details] > Patch I created the layout test fast/replaced/table-replaced-element.html is a scaled-down version of the page where the bug was originally found (http://support.google.com/ics/nexus/bin/answer.py?hl=en&answer=1637532). If you view fast/replaced/table-replaced-element.html in chromium, you will see that the presence of the percent height image causes an overflow of the table contents. This patch fixes the overflow problem, but does not respect the 60% height attribute of the image. (I have re-named this bug from "Percentage height replaced elements are sometimes displaying with incorrect size" to "Percentage height replaced elements sometimes cause overflow of table contents")
Eric Seidel (no email)
Comment 6 2012-05-15 13:48:55 PDT
Comment on attachment 141489 [details] Patch I'm happy to reivew this. But why do we need the wall of text? And does this need to be a pixel test? Can we use a dumpAsText or reference test instead? Can we use a div/divs with fixed heights instead of the wall fo text? inline-blocks will flow like text and you can control their size. r- because the test could be much better. Otherwise the chagne looks OK.
Terry Anderson
Comment 7 2012-05-18 12:04:48 PDT
Eric Seidel (no email)
Comment 8 2012-05-18 14:24:41 PDT
Comment on attachment 142759 [details] Patch Thanks.
WebKit Review Bot
Comment 9 2012-05-18 14:41:47 PDT
Comment on attachment 142759 [details] Patch Clearing flags on attachment: 142759 Committed r117633: <http://trac.webkit.org/changeset/117633>
WebKit Review Bot
Comment 10 2012-05-18 14:41:52 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.