| Summary: | REGRESSION (r29039): Scrollbar clipped passed visible region with fckeditor | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Alexander Mohr <amohr> | ||||||||
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||||
| Status: | NEW --- | ||||||||||
| Severity: | Normal | CC: | chrisb, ddkilzer, webkit | ||||||||
| Priority: | P1 | Keywords: | NeedsReduction, Regression | ||||||||
| Version: | 528+ (Nightly build) | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| URL: | http://www.fckeditor.net/demo | ||||||||||
| Bug Depends on: | |||||||||||
| Bug Blocks: | 9915 | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Alexander Mohr
2008-08-18 17:56:34 PDT
I've narrowed it down to happening somewhere between builds 28899 and 29350, but I'm blocked for narrowing it down further due to bug# 16804 :( Confirmed. A reduction of the HTML involved would be very useful. ok, after much build pain I found the change that causes this bug: 29039 (In reply to comment #3) > ok, after much build pain I found the change that causes this bug: 29039 This was a fix for Bug 15359. Created attachment 22913 [details]
Outer Frame
Created attachment 22914 [details]
Middle Frame
Created attachment 22915 [details]
Inner Frame
Comment on attachment 22913 [details] Outer Frame ><HTML style="border: thin solid blue; margin: 5px;"><HEAD> > ><SCRIPT> > function setHeight(str) > { > var iframe = document.getElementById('FCKeditor1___Frame'); > var iframeDoc = iframe.contentDocument; > var iframeElement = iframeDoc.getElementById('foo'); > iframeElement.style.height=str; > } ></SCRIPT> ></HEAD><BODY> ><BUTTON onmousedown="javascript:setHeight('100%')" type="button">100% height</BUTTON> ><BUTTON onmousedown="javascript:setHeight('75px')" type="button">75 height</BUTTON> ><TABLE width="100%" style="margin: 0pt;"> ><TBODY> ><TR><TD> ><DIV> > <IFRAME scrolling="no" style="border: thin solid blue; margin: 5px; width: 100%; height: 200px;" src="attachment.cgi?id=22914" id="FCKeditor1___Frame"> ></div> ></td></tr> ></tbody> ></table> ></body> ></html></IFRAME></DIV></TD></TR></TBODY></TABLE></BODY></HTML> (In reply to comment #8) > (From update of attachment 22913 [details] [edit]) my inline html editing skills on this site seem to be lacking ;) In any case, I've attached a testcase which I believe reproduces the source of the problem. I've structure the tree test files to have a separate color for each file. Furthermore, if the document has an iframe it gets another border with the same color. You'll note when opening the outer html, that the inner html's document's height is more that what is needed for the document (instead, set to it's intrinsic height) when the owner iframe was set to 100% height. It goes like this: Outer HTML->IFrame (200px height)->TR/TD(74px)+TR/TD/IFrame (100% height) I believe the expectation is that the IFrame would get sized to the remainder (200-borders/margins-74px) and not the intrinsic height. If you compare the behavior with firefox it seems firefox does not set the inner html size to it's container's iframe size, instead it gets set to the content size. We have also noted this issue recently. The strange fact is that it happens only if the editor has a reduced size (default btw). For example, you will reproduce the problem here: http://www.fckeditor.net/nightly/fckeditor/_samples/html/sample02.html But it will work well in this other sample instead: http://www.fckeditor.net/nightly/fckeditor/_samples/html/sample01.html Looking at the DOM with the Web Inspector, the only differences between sample01 and sample02, are the "width" attribute and style values for the outer iframe that holds the editor. The first is set to 300/300px, while we have 200/200px at the other. All the rest, including the inner tables and iframes have the same sizes (mostly 100% height in the inner stuff). It sounds like WebKit is forcing the inner iframe having a minimum height of exactly 150px (the computed style for the inner iframe in the Web Inspector). |