Bug 218090

Summary:	[css-logical] Fix getComputedStyle for logical properties
Product:	WebKit	Reporter:	Simon Fraser (smfr) <simon.fraser>
Component:	CSS	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	obrufau, simon.fraser, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	Safari Technology Preview
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=256739
Bug Depends on:
Bug Blocks:	185977

Description Simon Fraser (smfr) 2020-10-22 10:27:02 PDT

- Fix CSSOM problems like:
    document.documentElement.style.marginLeft = "1px";
    getComputedStyle(document.documentElement).marginLeft; // "1px", good
    document.documentElement.style.marginInlineStart = "2px";
    getComputedStyle(document.documentElement).marginLeft; // "2px", good
    document.documentElement.style.marginLeft = "3px";
    getComputedStyle(document.documentElement).marginLeft; // "2px", bad, should be "3px"

Comment 1 Radar WebKit Bug Importer 2020-10-22 10:27:21 PDT

<rdar://problem/70578989>

Comment 2 Oriol Brufau 2020-10-22 10:32:02 PDT

Note the underlying problem is not in getComputedStyle,

    document.documentElement.style.marginLeft = "1px";
    document.documentElement.style.cssText; // "margin-left: 1px;"
    document.documentElement.style.marginInlineStart = "2px";
    document.documentElement.style.cssText; // "margin-left: 1px; margin-inline-start: 2px;"
    document.documentElement.style.marginLeft = "3px";
    document.documentElement.style.cssText; // "margin-left: 3px; margin-inline-start: 2px;"

the last one should become "margin-inline-start: 2px; margin-left: 3px;" so that margin-left takes precedence.

Chromium has the same problem, Firefox does it right.

Comment 3 Oriol Brufau 2023-05-14 09:05:40 PDT

Fixed in bug 226461.

*** This bug has been marked as a duplicate of bug 226461 ***