19828 2008-06-30 11:17:51 -0700 Computed style for margin right is incorrect. 2009-03-02 15:42:23 -0800 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) PC Windows XP RESOLVED INVALID HasReduction P2 Normal --- 0 erik.arvidsson webkit-unassigned Humberto.Madeira pbakaus webkit oldest_to_newest 84760 0 erik.arvidsson 2008-06-30 11:17:51 -0700 The computed style value for margin right is not correct and it does not match Firefox nor Opera. el.style.marginRight = '10px'; document.defaultView.getComputedStyle(el, '').marginRight // != 10px The value returned seems to be related to the computed distance between the offsetParent right edge and the right border of the element. However it is not equal to: el.offsetParent.clientWidth - el.offsetLeft - el.offsetWidth Test case coming. 84761 1 22008 erik.arvidsson 2008-06-30 11:19:01 -0700 Created attachment 22008 The computed margin should be 10px Sometimes a reload is required to get the faulty margin value. 99931 2 pbakaus 2008-11-24 03:12:28 -0800 I can confirm this for all platforms. Additionally, this only happens if margin right is set through stylesheets - if set using inline styles, the correct value is reported. 104557 3 webkit 2009-01-06 05:09:40 -0800 CSS 2.1 specification says: "The following constraints must hold among the used values of the other properties: 'margin-left' + 'border-left-width' + 'padding-left' + 'width' + 'padding-right' + 'border-right-width' + 'margin-right' + scrollbar width (if any) = width of containing block" [...] If all of the above have a computed value other than 'auto', the values are said to be "over-constrained" and one of the used values will have to be different from its computed value. If the 'direction' property of the containing block has the value 'ltr', the specified value of 'margin-right' is ignored and the value is calculated so as to make the equality true. If the value of 'direction' is 'rtl', this happens to 'margin-left' instead." <http://www.w3.org/TR/CSS21/visudet.html#blockwidth> In my opinion WebKit correctly prints other than 10px *computed* value. Firefox is wrong in this case. Check attached test case. 104558 4 26458 webkit 2009-01-06 05:10:01 -0800 Created attachment 26458 test case 104559 5 webkit 2009-01-06 05:11:24 -0800 This bug report is invalid. 111914 6 Humberto.Madeira 2009-03-02 15:42:23 -0800 I believe this bug was rejected because the specification is being misread. The specification speaks of "computed values" and "used values" as being distinct from each other. e.g. in section 10.3 "(The value used for layout is sometimes referred to as the used value.) In principle, the values used are the same as the computed values, with 'auto' replaced by some suitable value, and percentages calculated based on the containing block, but there are exceptions." With respect to the constraints, they refer to the "used values" and not the computed values. e.g. in section 10.3.3 "The following constraints must hold among the used values of the other properties: 'margin-left' + 'border-left-width' + 'padding-left' + 'width' + 'padding-right' + 'border-right-width' + 'margin-right' + scrollbar width (if any) = width of containing block" So according to the specification, the computed values are under no constraints of any sort other than the usual CSS precedence rules. It is the "used values", which are used for layout purposes, that are constrained. I believe, after a closer look at the specification, that this bug is actually valid. 22008 2008-06-30 11:19:01 -0700 2008-06-30 11:19:01 -0700 The computed margin should be 10px temp3.html text/html 384 erik.arvidsson PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHRpdGxlPjwvdGl0bGU+CjxzdHlsZT4KCiN0 ZXN0IHsKICBiYWNrZ3JvdW5kOiByZWQ7CiAgd2lkdGg6IDEwZW07CiAgbWFyZ2luOiAxMHB4Owp9 Cgo8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5PgoKPGRpdiBpZD0idGVzdCI+SGk8L2Rpdj4KPGRpdiBp ZD0ib3V0Ij48L2Rpdj4KCjxzY3JpcHQ+Cgp2YXIgZWwgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJ ZCgndGVzdCcpOwp2YXIgb3V0ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ291dCcpOwpvdXQu aW5uZXJIVE1MID0gZG9jdW1lbnQuZGVmYXVsdFZpZXcuZ2V0Q29tcHV0ZWRTdHlsZShlbCwgJycp Lm1hcmdpblJpZ2h0OwoKPC9zY3JpcHQ+Cgo8L2JvZHk+CjwvaHRtbD4K 26458 2009-01-06 05:10:01 -0800 2009-01-06 05:10:01 -0800 test case test-page.html text/html 841 webkit PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHRpdGxlPjwvdGl0bGU+CjxzdHlsZT4KCiN0 ZXN0IHsKICBiYWNrZ3JvdW5kOiByZWQ7CiAgd2lkdGg6IDUwcHg7CiAgbWFyZ2luOiAxMHB4Owog Ym9yZGVyOiAwOwpwYWRkaW5nOjA7Cn0KCiNjb250IHtib3JkZXI6IDA7CnBhZGRpbmc6MDsKd2lk dGg6MTAwcHg7CmJhY2tncm91bmQtY29sb3I6IGJsdWU7fQo8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5 PgoKPGRpdiBpZD0iY29udCI+PGRpdiBpZD0idGVzdCI+SGk8L2Rpdj48L2Rpdj4KPGRpdiBpZD0i b3V0Ij48L2Rpdj4KPHA+J21hcmdpbi1sZWZ0JyArICdib3JkZXItbGVmdC13aWR0aCcgKyAncGFk ZGluZy1sZWZ0JyArICd3aWR0aCcgKyAncGFkZGluZy1yaWdodCcgKyAnYm9yZGVyLXJpZ2h0LXdp ZHRoJyArICdtYXJnaW4tcmlnaHQnICsgc2Nyb2xsYmFyIHdpZHRoIChpZiBhbnkpID0gd2lkdGgg b2YgY29udGFpbmluZyBibG9jayI8L3A+CjxwPjEwcHggKyAwICsgMCArIDUwcHggKyAwICsgMCAr IDQwIHB4IDxiPigxMCBweCBpcyBpZ25vcmVkIGFuZCB0aGUgdmFsdWUgaXMgY2FsY3VsYXRlZCBz byBhcyB0byBtYWtlIHRoZSBlcXVhbGl0eSB0cnVlKTwvYj4gPSAxMDBweCAtIHdpZHRoIG9mIGNv bnRhaW5pbmcgYmxvY2sgPC9wPgo8c2NyaXB0Pgp2YXIgZWwgPSBkb2N1bWVudC5nZXRFbGVtZW50 QnlJZCgndGVzdCcpOwp2YXIgb3V0ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ291dCcpOwpv dXQuaW5uZXJIVE1MID0gZG9jdW1lbnQuZGVmYXVsdFZpZXcuZ2V0Q29tcHV0ZWRTdHlsZShlbCwg JycpLm1hcmdpblJpZ2h0Owo8L3NjcmlwdD4KCjwvYm9keT4KPC9odG1sPg==