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

Support tab and arrow key navigation through dropdowns #1031

Closed
robomalo opened this issue Oct 7, 2016 · 8 comments
Closed

Support tab and arrow key navigation through dropdowns #1031

robomalo opened this issue Oct 7, 2016 · 8 comments
Labels

Comments

@robomalo
Copy link

robomalo commented Oct 7, 2016

Steps for Reproduction

  1. Visit http://quilljs.com/playground/
  2. Attempt to use tab key to select a headline format or normal
  3. The options are in <span> tags which cannot be focused

Expected behavior:

  1. Tab into format selector for heading, normal, etc.
  2. Hit enter or space bar
  3. Use tabs or arrow keys to select formatting
  4. Hit enter select formatting

Actual behavior:

Tabs pass over span tags because they cannot be focused

Platforms:

Everywhere

Version:

All versions

Note:

This actually failed our accessibility audit within LinkedIn for Slate

@scottsheffield
Copy link

If I'm remembering correctly, you guys are actually constructing the markup for the toolbar in Slate. Is that markup not usable if it doesn't use spans for the dropdown items, say if you switch to something focusable (button, whatever)?

@jhchen
Copy link
Member

jhchen commented Oct 11, 2016

Not sure Slate's exact implementation but the default picker does create <span>s. The quick solution is to change the label <span> into a <a> but then the obvious next Issue report would be inability to down arrow and select. So a satisfactory resolution to this Issue imo is a more full featured implementation.

@robomalo
Copy link
Author

Anchors aren't totally semantically correct. Would it be possible to
control it with the actual select form item?
On Mon, Oct 10, 2016 at 10:34 PM Jason Chen [email protected]
wrote:

Not sure Slate's exact implementation but the default picker does create
s. The quick solution is to change the label into a but
then the obvious next Issue report would be inability to down arrow and
select.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1031 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACMQnX9Puh04YXosMDS48pai4XwdutLks5qyx_KgaJpZM4KRU_c
.

@jhchen
Copy link
Member

jhchen commented Oct 12, 2016

What is? A button? I'd rather get it right then make it configurable so other people can get it wrong.

@robomalo
Copy link
Author

Buttons would be correct in the presentation side but I was wondering if we
could do something with the actual select element already in the page.
Isn't there one?
On Tue, Oct 11, 2016 at 11:25 PM Jason Chen [email protected]
wrote:

What is? A button? I'd rather get it right then make it configurable so
other people can get it wrong.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1031 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACMQqJShpQSJwUfXHCGToOMH1opKBh0ks5qzH1cgaJpZM4KRU_c
.

@jhchen
Copy link
Member

jhchen commented Oct 12, 2016

There is but the current state of HTML technology does not allow us to style <select> and <option>

@robomalo
Copy link
Author

Oh, I know... I was thinking like positioning it off the page with CSS but
still have it in the markup to be controlled by tabs. But then that
wouldn't help with the focused elements on the screen. Seems like no
getting around the buttons maybe.
On Tue, Oct 11, 2016 at 11:55 PM Jason Chen [email protected]
wrote:

There is but the current state of HTML technology does not allow us to
style and — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com//issues/1031#issuecomment-253134040, or mute the thread https://github.com/notifications/unsubscribe-auth/AACMQmm6kKrzxRfvKmqSzhHZQl2Ccf0Wks5qzIRcgaJpZM4KRU_c .

@jhchen jhchen changed the title Accessibility-keyboard tabbing issues with .ql-headline Support tab and arrow key navigation through dropdowns Oct 20, 2016
@jhchen jhchen added the feature label Oct 20, 2016
@quill-bot
Copy link

Quill 2.0 has been released (announcement post) with many changes and fixes. If this is still an issue please create a new issue after reviewing our updated Contributing guide 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants