Closed Bug 1987957 Opened 4 months ago Closed 4 months ago

Handle removal of last successful position

Categories

(Core :: CSS Parsing and Computation, enhancement)

enhancement
Points:
2

Tracking

()

RESOLVED FIXED
145 Branch
Tracking Status
firefox145 --- fixed

People

(Reporter: dshin, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

(Whiteboard: [anchorpositioning:m2])

Attachments

(1 file)

No description provided.

See spec:
Need to clear last successful position when:

  • el is not absolutely positioned
  • el’s computed value for position-try-fallbacks has changed
  • Any of the @position-try rules referenced by el’s position-try-fallbacks have been added, removed, or mutated.

Last item should be separate, handled by bug 1987960 and bug 1962598

Points: --- → 2
Whiteboard: [anchorpositioning:triage] → [anchorpositioning:m2]
Depends on: 1962598

This should cause the last-fallback to be cleared on any change to the list,
not only if the current index is out-of-range.

(To be on the safe side, for now I've left the existing length check in place
in AbsoluteContainingBlock::ReflowAbsoluteFrame, with an UNREACHABLE assertion,
though we may eventually want to remove it.)

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Pushed by jkew@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/4e216c149b31 https://hg.mozilla.org/integration/autoland/rev/76d615f2488f Remove any recorded last-successful-fallback if the fallback list changes. r=layout-reviewers,emilio,dshin
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 145 Branch
QA Whiteboard: [qa-triage-done-c146/b145]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: