Closed Bug 1902325 Opened 1 year ago Closed 1 year ago

Remove z-index concept from views.

Categories

(Core :: Layout, task)

task

Tracking

()

RESOLVED FIXED
129 Branch
Tracking Status
firefox129 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

No description provided.

This comes back to ~since forever, since absolutely positioned elements
used to have views. But now is basically only plumbed from
nsMenuPopupFrame to nsIWidget.

The main issue is that that concept is basically fiction nowadays:

  • Wayland can't reorder popups.
  • macOS doesn't implement it because SetZIndex calls PlaceBehind which
    only has a windows impl.
  • Unclear whether it works properly on Windows as well.

Also, the front-end never sets it anyways.

So remove this fiction to simplify relevant code. We might want to
provide a z-ordering of popups somehow else, but it should probably live
in nsXULPopupManager etc rather than nsView.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fc3e01bf7b80 Remove zIndex concept from views. r=tnikkel,layout-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: