20442 2008-08-18 17:56:34 -0700 REGRESSION (r29039): Scrollbar clipped passed visible region with fckeditor 2008-08-22 00:12:44 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All NEW http://www.fckeditor.net/demo NeedsReduction, Regression P1 Normal --- 9915 1 amohr webkit-unassigned chrisb ddkilzer webkit oldest_to_newest 88786 0 amohr 2008-08-18 17:56:34 -0700 If you go to the editor area you'll first notice that the bottom line is clipped. Next, if you hit return a few times with the cursor in the editable area you'll note that the scrollbar pops up clipped. This worked correctly in Safari 3.0.4 and broke starting with Safari 3.1. 88795 1 amohr 2008-08-18 20:53:31 -0700 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 :( 88839 2 ddkilzer 2008-08-19 12:51:18 -0700 Confirmed. A reduction of the HTML involved would be very useful. 88864 3 amohr 2008-08-19 16:23:26 -0700 ok, after much build pain I found the change that causes this bug: 29039 88877 4 ddkilzer 2008-08-19 17:15:05 -0700 (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. 88979 5 22913 amohr 2008-08-20 18:05:38 -0700 Created attachment 22913 Outer Frame 88980 6 22914 amohr 2008-08-20 18:06:12 -0700 Created attachment 22914 Middle Frame 88981 7 22915 amohr 2008-08-20 18:06:25 -0700 Created attachment 22915 Inner Frame 88982 8 22913 amohr 2008-08-20 18:08:30 -0700 Comment on attachment 22913 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> 88984 9 amohr 2008-08-20 18:42:10 -0700 (In reply to comment #8) > (From update of attachment 22913 [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. 89078 10 webkit 2008-08-22 00:07:26 -0700 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). 22913 2008-08-20 18:05:38 -0700 2008-08-20 18:05:38 -0700 Outer Frame outer-outer-frame.html text/html 773 amohr PGh0bWwgc3R5bGU9ImJvcmRlcjogdGhpbiBzb2xpZCBibHVlOyBtYXJnaW46IDVweDsiPg0KPGhl YWQ+DQo8c2NyaXB0Pg0KCWZ1bmN0aW9uIHNldEhlaWdodChzdHIpDQoJew0KCQl2YXIgaWZyYW1l ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ0ZDS2VkaXRvcjFfX19GcmFtZScpOw0KCQl2YXIg aWZyYW1lRG9jID0gaWZyYW1lLmNvbnRlbnREb2N1bWVudDsNCgkJdmFyIGlmcmFtZUVsZW1lbnQg PSBpZnJhbWVEb2MuZ2V0RWxlbWVudEJ5SWQoJ2ZvbycpOw0KCQlpZnJhbWVFbGVtZW50LnN0eWxl LmhlaWdodD1zdHI7DQoJfQ0KPC9zY3JpcHQ+DQo8L2hlYWQ+DQo8Ym9keT4NCjxidXR0b24gdHlw ZT0iYnV0dG9uIiBvbm1vdXNlZG93bj0iamF2YXNjcmlwdDpzZXRIZWlnaHQoJzEwMCUnKSI+MTAw JSBoZWlnaHQ8L2J1dHRvbj4NCjxidXR0b24gdHlwZT0iYnV0dG9uIiBvbm1vdXNlZG93bj0iamF2 YXNjcmlwdDpzZXRIZWlnaHQoJzc1cHgnKSI+NzUgaGVpZ2h0PC9idXR0b24+DQo8dGFibGUgd2lk dGg9IjEwMCUiIHN0eWxlPSdtYXJnaW46MCc+DQo8dGJvZHk+DQo8dHI+PHRkPg0KPGRpdj4NCiAg ICA8aWZyYW1lIGlkPSJGQ0tlZGl0b3IxX19fRnJhbWUiIHNyYz0ib3V0ZXItZnJhbWUuaHRtbCIg c2Nyb2xsaW5nPSJubyIgc3R5bGU9IndpZHRoOjEwMCU7IGhlaWdodDoyMDBweDsgYm9yZGVyOiB0 aGluIHNvbGlkIGJsdWU7IG1hcmdpbjogNXB4OyI+DQo8L2Rpdj4NCjwvdGQ+PC90cj4NCjwvdGJv ZHk+CQ0KPC90YWJsZT4NCjwvYm9keT4NCjwvaHRtbD4= 22914 2008-08-20 18:06:12 -0700 2008-08-20 18:06:12 -0700 Middle Frame outer-frame.html text/html 379 amohr PGh0bWwgc3R5bGU9ImJvcmRlcjogdGhpbiBzb2xpZCByZWQ7IG1hcmdpbjogNXB4OyI+DQo8aGVh ZD48L2hlYWQ+DQo8Ym9keT4NCjx0YWJsZSBzdHlsZT0id2lkdGg6IDEwMCUiPg0KCTx0Ym9keT4N Cgk8dHIgPg0KCQk8dGQgc3R5bGU9ImhlaWdodDo3NHB4Ij50b29sIGJhciBoZXJlPC90ZD4NCgk8 L3RyPg0KCTx0cj4NCiAgICAgICAgPHRkPg0KICAgICAgICAgICAgPGlmcmFtZSBpZD0iZm9vIiBz dHlsZT0iaGVpZ2h0OiAxMDAlOyB3aWR0aDoxMDAlOyBib3JkZXI6IHRoaW4gc29saWQgcmVkOyBt YXJnaW46IDVweDsiIHNyYz0iaW5uZXItZnJhbWUuaHRtbCI+DQoJCTwvdGQ+DQoJPC90cj4NCgk8 L3Rib2R5Pg0KPC90YWJsZT4NCjwvYm9keT4NCjwvaHRtbD4NCg== 22915 2008-08-20 18:06:25 -0700 2008-08-20 18:06:25 -0700 Inner Frame inner-frame.html text/html 241 amohr PGh0bWwgc3R5bGU9ImJvcmRlcjogdGhpbiBzb2xpZCBncmVlbjsgcGFkZGluZzogMHB4OyBtYXJn aW46IDVweDsiPg0KPGhlYWQ+DQo8c2NyaXB0Pg0KZG9jdW1lbnQuZGVzaWduTW9kZSA9ICJvbiI7 DQo8L3NjcmlwdD4NCjwvaGVhZD4NCjxib2R5IGlkPSJ0aGVib2R5IiBkaXI9Imx0ciI+DQoJVGhp cyBpcyBzb21lIDxzdHJvbmc+IHNhbXBsZSB0ZXh0PC9zdHJvbmc+Ljxicj48YnI+PGJyPg0KPC9i b2R5Pg0KPC9odG1sPg==