Closed
Bug 2003593
Opened 4 days ago
Closed 3 days ago
Changing default anchor to another element doesn't invalidate positioned element if it has `transform`
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
147 Branch
| Tracking | Status | |
|---|---|---|
| firefox147 | --- | fixed |
People
(Reporter: dshin, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
See Bug 2003544 Comment 1 for testcase.
|
Assignee | |
Comment 1•4 days ago
|
||
It's not quite about transform right? I don't see what invalidates on anchor-name changes.
|
|
Assignee | |
Comment 2•4 days ago
|
||
|
||
Updated•4 days ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
|
Reporter | |
Comment 3•4 days ago
•
|
||
I'd hope we hit this - We're getting anchor using GetAnchor into that call instead of using the cached value. Disabling transform in DevTools does make a difference as well.
|
|
Assignee | |
Comment 4•4 days ago
|
||
Ohh, I see, sorry, totally forgot about that code. Let me see what's going on.
|
|
||
Updated•4 days ago
|
Attachment #9530425 -
Attachment description: Bug 2003593 - Add invalidation for anchor-name changes. r=dshin,#anchor-pos → Bug 2003593 - Invalidate for anchor changes correctly. r=dshin,#anchor-pos
Pushed by ealvarez@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/282a5bc016f3
https://hg.mozilla.org/integration/autoland/rev/dfdab599fd04
Invalidate for anchor changes correctly. r=dshin,layout-anchor-positioning-reviewers,firefox-style-system-reviewers,layout-reviewers
|
||
Comment 6•3 days ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 days ago
status-firefox147:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 147 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/56449 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
You need to log in
before you can comment on or make changes to this bug.
Description
•