Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Popup spec] Specify hide event sequence #324

Closed
melanierichards opened this issue Apr 9, 2021 · 2 comments
Closed

[Popup spec] Specify hide event sequence #324

melanierichards opened this issue Apr 9, 2021 · 2 comments
Labels
popover The Popover API popover-spec Spec-related issues for the Popover API stale

Comments

@melanierichards
Copy link
Collaborator

On MicrosoftEdge/MSEdgeExplainers#450, @JanMiksovsky said:

I think it'd be helpful to add an example describing the sequence of events raised for the hide operation, in the same way the KeyboardEvent sequence documents the result of a key press. The event sequence could indicate:

  • Relative ordering of blur and hide events. For popups that can have the focus, I assume they'd see blur occur first, then hide?
  • Relative ordering of events across a stack of popups being dismissed. I assume the topmost popup gets the (non-bubbling?) hide event first, then the next one, up the hierarchy.
  • Relative ordering of hide events and the actual removal of the popup from view: do all popups being hidden get the hide event, then the popups disappear?

On this topic, I notice that HTMLDialogElement doesn't define a show event, but that doesn't mean one wouldn't be useful. Document, for example, defines events like load and visibilitychange that can be used when a document is opened, so I thought some consideration could be given to defining a show event for popups.

@github-actions
Copy link

There hasn't been any discussion on this issue for a while, so we're marking it as stale. If you choose to kick off the discussion again, we'll remove the 'stale' label.

@github-actions github-actions bot added the stale label Mar 19, 2022
@mfreed7 mfreed7 changed the title [Popup] Specify hide event sequence [Popup spec] Specify hide event sequence May 13, 2022
@mfreed7 mfreed7 added the popover-spec Spec-related issues for the Popover API label May 13, 2022
@mfreed7
Copy link
Collaborator

mfreed7 commented Aug 24, 2023

I believe this issue has been handled via the landed popover spec. I'd like to close it. If you disagree, please feel free to re-open with a comment about what's still missing.

@mfreed7 mfreed7 closed this as completed Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
popover The Popover API popover-spec Spec-related issues for the Popover API stale
Projects
None yet
Development

No branches or pull requests

2 participants