| Summary: | iframe does not respect container height in strict and transitional doctypes | ||
|---|---|---|---|
| Product: | [Applications] konqueror | Reporter: | Martin van Es <bugs> |
| Component: | khtml | Assignee: | Konqueror Developers <konq-bugs> |
| Status: | RESOLVED NOT A BUG | ||
| Severity: | normal | CC: | browserbugs2 |
| Priority: | NOR | ||
| Version: | 3.5 | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
|
Description
Martin van Es
2008-10-10 17:39:13 UTC
Martin van Es, 1- Can you attach a reduced, minimized testcase? "Throw out anything that has nothing to do with the bug (...) The important part is: the page should be as short as possible to be able to still reproduce the bug." http://www.konqueror.org/investigatebug/#html Since your code involves the deprecated iframe element, please use this doctype declaration <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> 2- Can you specify in a formal manner what is the expected result, the actual result, the browser details (Firefox version, Konqueror version, platform, os, rendering modes, doctype declarations, etc.) which are relevant to your bug report. 3- As far as I can see, you code creates a constraining and constrained design (height: 100% is declared, including for a single table cell) where a vertical scrollbar should be generated to render both rows. 4- Please explain why border-collapse is declared or required or needed in your code 5- Note that all browsers have a default padding for table cells: Konqueror as well as Firefox (all versions). On the other hand, html, body, table elements have a default padding of 0. 6- iframe is an inline replaced element without an intrinsic ratio. There are rules that apply in such cases (§ 10.3.2, § 10.6.2). Gérard Martin van Es, I have good news regarding this bug report. > 1- Can you attach a reduced, minimized testcase? 2 reduced testcases: Iframe element http://www.gtalbot.org/BrowserBugsSection/Konqueror4Bugs/Bug172525-iframe-height.html (rendered in Almost standards rendering mode) Embedded HTML document with the HTML4 <object> element http://www.gtalbot.org/BrowserBugsSection/Konqueror4Bugs/Bug172525-object-height.html (rendered in web standards compliant rendering mode) > 2- Can you specify in a formal manner what is the expected result, the actual > result, the browser details (Firefox version, Konqueror version, platform, os, > rendering modes, doctype declarations, etc.) which are relevant to your bug > report. I re-read the bug report and expected results would be to *not* have, not see a vertical scrollbar in the webpage. > 3- As far as I can see, you code creates a constraining and constrained design > (height: 100% is declared, including for a single table cell) where a vertical > scrollbar should be generated to render both rows. The important, decisive information is given in CSS 2.1 spec, section 17.5.3 Table height algorithms which says " CSS 2.1 does not define how the height of table cells and table rows is calculated when their height is specified using percentage values. " http://www.w3.org/TR/CSS21/tables.html#height-layout And this is exactly what happens too in all tested browsers. Firefox 6.0.2, Chrome 14.0.835.186 and Safari 5.1 do not render a vertical scrollbar in the 2 testcases. Konqueror 4.7.0, Opera 11.51 and Internet Explorer 8 render a vertical scrollbar in the 2 testcases, which allows to scroll the additional 80px of document box height. There is no right or wrong rendering in such testcases actually. I am using KDE Platform Version: 4.7.0 Konqueror version: 4.7.0 (KHTML rendering engine) Qt Version: 4.7.2 Operating System: Linux 2.6.38-11-generic-pae i686 (32bits) Distribution: Kubuntu 11.04 here. > 4- Please explain why border-collapse is declared or required or needed in > your code In border-collapse: separate, browsers all render a default border-spacing of 2px (horizontal and vertical). One way that very often people neutralize, zero such border-spacing between table cells is to set 'border-collapse: collapse'. A lot of people do that to neutralize small vertical and horizontal gaps which they may not understand the origin. So, even though there is no border around any part of the table, the side effects of 'border-collapse: collapse' work. > 5- Note that all browsers have a default padding for table cells: Konqueror as > well as Firefox (all versions). On the other hand, html, body, table elements > have a default padding of 0. In the 2 provided testcases, this has been done. ---------- So, to sum up all this, this bug report in my opinion should be resolved as INVALID. The rendered layout in Konqueror 4.7.0 is compliant with the CSS 2.1 spec. regards, Gérard Resolving as INVALID |