Skip to content

Set taborder for TOTP Verification - #3068

Merged
ssddanbrown merged 2 commits into
BookStackApp:masterfrom
raccettura:patch-1
Nov 22, 2021
Merged

ssddanbrown merged 2 commits into
BookStackApp:masterfrom
raccettura:patch-1

Conversation

@raccettura

Copy link
Copy Markdown
Contributor

Adding tabindex=0 means when pressing tab the focus goes right to the TOTP input field. When using a Password Manager this makes it easier than having to hit tab 3X to get the right focus.

Adding tabindex=0 means when pressing tab the focus goes right to the TOTP input field.  When using a Password Manager this makes it easier than having to hit tab 3X to get the right focus.
@ssddanbrown

Copy link
Copy Markdown
Member

Thanks for the PR @raccettura.

Adding tabindex=0 means when pressing tab the focus goes right to the TOTP input field.

Have you tested this? From my understanding, and testing of this PR, this is not the case.

Even if this PR did change the tabbing order I try not to alter the tab order to ensure we're causing an odd accessibility experience. I think we've used an autofocus attribute elsewhere. Might be more apt here, assuming no accessibility concerns.

@raccettura

Copy link
Copy Markdown
Contributor Author

Seemed to work for me in Chrome. But you make some good points to prefer autofocus so updated the PR to reflect that.

@ssddanbrown ssddanbrown added this to the v21.11.1 milestone Nov 22, 2021
@ssddanbrown
ssddanbrown merged commit 37db51a into BookStackApp:master Nov 22, 2021
@ssddanbrown

Copy link
Copy Markdown
Member

Thanks @raccettura. Merged for next patch release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants