Closed Bug 1733155 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 30948 - Remove "resize" as a <popup> light dismiss trigger

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox95 fixed)

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 30948 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/30948
Details from upstream follow.

Mason Freed <masonf@chromium.org> wrote:

Remove "resize" as a <popup> light dismiss trigger

This ended up being a confusing part of the implementation of
\<popup>: it was easy to accidentally resize the popup, causing
it to light dismiss. For example, adding a :hover style that
adds a border to the popup or an element might cause a light
dismiss. And the rationale for this being a trigger in the
first place was an implementation concern - avoiding
expensive relayout in the case that the popup is anchor-
positioned to something on the page.

Bug: 1168738
Change-Id: Ic5a3e616941ace8b47b4abc02dbf786aa78521c4
Fixed: 1252176
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3180152
Commit-Queue: Mason Freed \<masonf@chromium.org>
Auto-Submit: Mason Freed \<masonf@chromium.org>
Reviewed-by: Dan Clark \<daniec@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#924911}

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/58ae100c2749
[wpt PR 30948] - Remove "resize" as a <popup> light dismiss trigger, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.