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] Move invoker activation behaviors to other parts of the HTML spec #390

Closed
melanierichards opened this issue Aug 27, 2021 · 2 comments
Labels
needs edits This is ready for edits to be made popover The Popover API popover-spec Spec-related issues for the Popover API stale

Comments

@melanierichards
Copy link
Collaborator

melanierichards commented Aug 27, 2021

#355 lands a first editor's draft of <popup> in Open UI, and is intended to be brought to various parts of the HTML spec in WHATWG.

Of the steps in the "Showing/hiding a popup element via the popup attribute" section, @domenic's advice was to:

  • Add/modify explicit activation behavior for button and input in the various relevant states. Button element example. Having a single algorithm multiple places call makes sense.
  • Modify the focus update steps:

I think the focus update steps need modification for "OR the invoker is an input in the email, number, search, tel, text, or url states and focus is set to the invoker". That is, whenever focus is updated for an input type=email, you need to check if that input type=email is the invoker for a popup and then run these steps if so.

Opening this issue to track the edit as suggested. We can add a snippet to the Open UI spec as I've done here: https://open-ui.org/components/popup#html-incubation-text-user-interaction-focus-processing-model

@melanierichards melanierichards added needs edits This is ready for edits to be made popover The Popover API labels Aug 27, 2021
@github-actions
Copy link

github-actions bot commented Mar 4, 2022

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 4, 2022
@mfreed7 mfreed7 changed the title [Popup] Move invoker activation behaviors to other parts of the HTML spec [Popup spec] Move invoker activation behaviors to other parts of the HTML spec May 13, 2022
@mfreed7 mfreed7 added the popover-spec Spec-related issues for the Popover API label May 13, 2022
@josepharhar
Copy link
Collaborator

My HTML spec PR hooks into activation behavior. This issue can be closed

@mfreed7 mfreed7 closed this as completed Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs edits This is ready for edits to be made popover The Popover API popover-spec Spec-related issues for the Popover API stale
Projects
None yet
Development

No branches or pull requests

3 participants