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

Hiding an element should remove it from the focus tree #57

Closed
viciious opened this issue Oct 12, 2019 · 5 comments
Closed

Hiding an element should remove it from the focus tree #57

viciious opened this issue Oct 12, 2019 · 5 comments

Comments

@viciious
Copy link
Contributor

This is somewhat a continuation of issue #40. Consider the following scenario:

  1. A document is shown and it has a bunch of elements with the "autofocus" attribute.
  2. The first element is focused as it should.
  3. The OnShow script then decides it doesn't want that first element to be displayed and hides by setting "display" to "none".
  4. Pressing ENTER will result in a click event on the hidden element.
@mikke89
Copy link
Owner

mikke89 commented Oct 12, 2019

Does this work for you?

@viciious
Copy link
Contributor Author

viciious commented Oct 12, 2019

Yes, sort of, thanks. The next question is: should, in case of such event, the next autofocus element be focused?

@mikke89
Copy link
Owner

mikke89 commented Oct 12, 2019

I don't know. What do web browsers do? It's not really something I want to spend time on :P

@viciious
Copy link
Contributor Author

You're right, let's keep it the way it is.

@mikke89
Copy link
Owner

mikke89 commented Oct 12, 2019

Cool :) I guess we can consider this fixed then.

@mikke89 mikke89 closed this as completed Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants