14722 2007-07-23 07:51:46 -0700 REGRESSION: border is not taken in account with offsetLeft (and probally offsetTop) 2007-08-02 13:45:06 -0700 1 1 1 Unclassified WebKit Layout and Rendering 523.x (Safari 3) All All RESOLVED WONTFIX HasReduction, InRadar, Regression P1 Major --- 1 sjoerdmulder webkit-unassigned bdakin oldest_to_newest 3969 0 sjoerdmulder 2007-07-23 07:51:46 -0700 See attached testcase where it should set the offsetLeft value with the size of the 50px border of the parent it does not take this in account. In Safari 2.0.4 it works correctly, tested on Windows XP, Mac OSX 10.4.10 and Mac OSX 10.5 3968 1 15649 sjoerdmulder 2007-07-23 07:53:21 -0700 Created attachment 15649 Testcase 3865 2 ddkilzer 2007-07-23 23:04:46 -0700 (In reply to comment #1) > Created an attachment (id=15649) [edit] > Testcase So the test case passes if "border - left:78" is printed, but fails if "border - left:28" is printed? Firefox 2.0.0.5 and Safari 3.0 (522.12) + WebKit r24534 print "border - left:28". Opera 9.21 and Safari 2.0.4 (419.3) + original WebKit print "border -left:78". What does MSIE 6/7 print? 3861 3 ddkilzer 2007-07-23 23:05:55 -0700 Confirming difference between Safari 2.0 and Safari 3.0 + ToT WebKit, but I'm not sure if this is a regression or not. (Assuming it is for now.) 3852 4 sjoerdmulder 2007-07-24 00:53:34 -0700 I Agree that the change isnt a regression if this was intented. The problem is part of a bigger issue: measurement of layout, - IE has getBoundingClientRect() (offsetTop / offsetLeft is not reliable) - Gecko has getBoxObjectFor() (offsetTop / offsetLeft is not reliable) - Opera / Webkit have offsetTop / offsetLeft / offsetParent working reliable. In Safari 2 and Opera 9 the behavior is like expected, the 'offsetLeft' of the elements summed together is the value in pixels the element is positioned from the left side of the screen (78px). In Safari 3 this changed, sure their can be an exception added for Safari 3 but it would be quite annoying if in Safari 3.0.1. the behavior is different again,. That's why i reporting this bug as a regression, a lot of frameworks / toolkits which use 'offset' properties to measure position need to implement specific Safari 3 code 3827 5 ddkilzer 2007-07-24 07:09:49 -0700 (In reply to comment #2) > So the test case passes if "border - left:78" is printed, but fails if "border > - left:28" is printed? Yes, per Comment #4. 3825 6 ddkilzer 2007-07-24 07:10:56 -0700 <rdar://problem/5356797> 3822 7 ddkilzer 2007-07-24 07:36:53 -0700 Bug 13237, Comment #10: Comment #10 From Dave Hyatt 2007-07-24 04:08 PDT [reply] offsetLeft/Top are relative to your offsetParent. I would guess we had offsetParent wrong. The goal here is to match IE. What does IE return here? 3813 8 sjoerdmulder 2007-07-24 08:56:51 -0700 I think the behavior of IE should not be matched with offsetParent / left / top. IE is complete buggy concerning to offsetParent / Left / Top (e.g. when mixing border / padding / margin / box-model / position / overflow) and though not reliable. Better would be to keep matching Safari 2.0.4 (was working fine except for TR elements but that is solved) and Opera 9.x (i remember a discussion on the Opera lists a while ago, see: http://dump.testsuite.org/2006/dom/style/offset/spec) 3132 9 koivisto 2007-08-02 10:56:20 -0700 In Safari 3.0 we are matching Firefox offset behavior on purpose, see bug 12687. I think this should be changed now. 3131 10 koivisto 2007-08-02 10:57:08 -0700 I meant to say that this should _not_ be changed now. 3120 11 sullivan 2007-08-02 13:45:06 -0700 In Radar, Dave Hyatt agrees that this should not be changed. Setting to WONTFIX 15649 2007-07-23 07:53:21 -0700 2007-07-23 07:53:21 -0700 Testcase index.html text/html 883 sjoerdmulder PCEtLSAtLT4KPCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjEvL0VO IiAiaHR0cDovL3d3dy53My5vcmcvVFIveGh0bWwxMS9EVEQveGh0bWwxMS5kdGQiPgo8aHRtbCB4 bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCI+Cgk8aGVhZD4KCQk8dGl0bGU+YnVn IDAwMDA8L3RpdGxlPgoJCTxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KCQkJZnVuY3Rp b24gZWxvYWQoKXsKCQkJCXZhciBvYmogPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgndGVzdCcp OwoJCQkJb2JqLmFwcGVuZENoaWxkKCBkb2N1bWVudC5jcmVhdGVUZXh0Tm9kZSgnYm9yZGVyIC0g bGVmdDonICsgZ2V0TGVmdChvYmopICkpOwoJCQl9CgkJCWZ1bmN0aW9uIGdldExlZnQob0VsbSl7 CgkJCQl2YXIgaUxlZnQgPSAwOwoJCQkJd2hpbGUob0VsbSl7CgkJCQkJaUxlZnQgKz0gb0VsbS5v ZmZzZXRMZWZ0OwoJCQkJCW9FbG0gPSBvRWxtLm9mZnNldFBhcmVudDsKCQkJCX0KCQkJCXJldHVy biBpTGVmdDsKCQkJfQoJCTwvc2NyaXB0PgoJPC9oZWFkPgoJPGJvZHkgb25sb2FkPSJlbG9hZCgp OyI+CgkJPGgxPmJ1ZyAwMDAwPC9oMT4KCQk8ZGl2IHN0eWxlPSJwb3NpdGlvbjpyZWxhdGl2ZTti YWNrZ3JvdW5kLWNvbG9yOiNGOTY7d2lkdGg6MzAwcHg7aGVpZ2h0OjIwMHB4O2JvcmRlcjpkb3R0 ZWQgNTBweCBsaW1lO3BhZGRpbmc6NXB4OyI+CgkJCTxkaXYgc3R5bGU9InBvc2l0aW9uOmFic29s dXRlO2xlZnQ6MjBweDt0b3A6MjVweDtiYWNrZ3JvdW5kLWNvbG9yOiM5RjY7d2lkdGg6MTAwcHg7 aGVpZ2h0OjEwMHB4O2JvcmRlcjpzb2xpZCAzMHB4ICM5OUM7IiBpZD0idGVzdCI+CgkJCTwvZGl2 PgoJCTwvZGl2PgoJPC9ib2R5Pgo8L2h0bWw+Cg==