18203 2008-03-28 19:08:24 -0700 Right floats should be allowed to overflow past left border edge 2008-09-03 11:32:19 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED http://css-class.com/test/css/overflow/float-container-margin-overflow.htm P2 Normal --- 1 hyatt hyatt alan oldest_to_newest 75718 0 hyatt 2008-03-28 19:08:24 -0700 See: http://css-class.com/test/css/overflow/float-container-margin-overflow.htm In current WebKit, the right float is not allowed to have its left edge extend beyond the left border edge. This is not correct according to the CSS2.1 spec and does not match Firefox. 75719 1 20179 hyatt 2008-03-28 19:09:47 -0700 Created attachment 20179 Remove the clamping code Remove the code that clamps right floats to the left border edge. 75720 2 hyatt 2008-03-28 19:11:26 -0700 I will land this patch with a better test case that doesn't depend on resizing the window. 75879 3 20179 darin 2008-03-30 16:44:24 -0700 Comment on attachment 20179 Remove the clamping code r=me because I know you'll add a ChangeLog and test case -- but why not do that before setting the review flag next time? 75884 4 hyatt 2008-03-30 17:07:15 -0700 Because keeping a ChangeLog in my tree for days is a giant pain when I'm doing other work. 76427 5 alan 2008-04-06 10:49:20 -0700 I have created a new test case without the need for scrolling. Just one element floated right. Firefox will hide the left edge of the float beyond the left edge of the viewpoint. Safari will show the full width of the float causing a horizontal scrollbar. See: http://css-class.com/test/bug/float-right-left-edge-hidden.htm 82635 6 darin 2008-06-08 12:27:32 -0700 Hyatt, are you planning on landing this? 89987 7 20179 hyatt 2008-09-03 00:11:35 -0700 Comment on attachment 20179 Remove the clamping code Let me get a test case and changelog. 90041 8 hyatt 2008-09-03 11:32:19 -0700 Fixed in r36061. 20179 2008-03-28 19:09:47 -0700 2008-09-03 00:11:35 -0700 Remove the clamping code patch.txt text/plain 585 hyatt SW5kZXg6IHJlbmRlcmluZy9SZW5kZXJCbG9jay5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcmVuZGVyaW5n L1JlbmRlckJsb2NrLmNwcAkocmV2aXNpb24gMzEzNzEpCisrKyByZW5kZXJpbmcvUmVuZGVyQmxv Y2suY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yMjI5LDcgKzIyMjksNiBAQCBib29sIFJlbmRlckJs b2NrOjpwb3NpdGlvbk5ld0Zsb2F0cygpCiAgICAgICAgICAgICAgICAgeSArPSBtaW4oaGVpZ2h0 UmVtYWluaW5nTGVmdCwgaGVpZ2h0UmVtYWluaW5nUmlnaHQpOwogICAgICAgICAgICAgICAgIGZ4 ID0gcmlnaHRSZWxPZmZzZXQoeSwgcm8sIGZhbHNlLCAmaGVpZ2h0UmVtYWluaW5nUmlnaHQpOwog ICAgICAgICAgICAgfQotICAgICAgICAgICAgZnggPSBtYXgoZi0+bV93aWR0aCwgZngpOwogICAg ICAgICAgICAgZi0+bV9sZWZ0ID0gZnggLSBmLT5tX3dpZHRoOwogICAgICAgICAgICAgby0+c2V0 UG9zKGZ4IC0gby0+bWFyZ2luUmlnaHQoKSAtIG8tPndpZHRoKCksIHkgKyBvLT5tYXJnaW5Ub3Ao KSk7CiAgICAgICAgIH0K