Add focus for a valid element in lists support

ai / keyux

JS library to improve keyboard UI of web apps

https://ai.github.io/keyux/

MIT License

391 stars 18 forks source link

Add focus for a valid element in lists support #13

Closed ignatiqq closed 6 months ago

ignatiqq commented 6 months ago

solves https://github.com/ai/keyux/issues/11

ai commented 6 months ago

Can you explain the use case?

ai commented 6 months ago

If you are talking about https://github.com/ai/keyux/issues/11 task, note that it introduce custom data- attributes. This is why you should not use tagName.

ignatiqq commented 6 months ago

I'm sorry. I need help. If we focused on element which is disabled. But we have element which has both valid attrs: 'data-keyux-ignore-hotkeys' 'data-keyux-hotkeys'

should we click on it. Or we should only take care of activeElement "data-keyux-hotkeys"?

ai commented 6 months ago

Can you write me to https://t.me/sitnik ? I think the chat will be better to explain the idea.