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

[Bug Report] V-dialog does not retain focus #9144

Closed
avenmore opened this issue Sep 28, 2019 · 3 comments · Fixed by socialsoftware/quizzes-tutor#52, socialsoftware/quizzes-tutor#55 or drakeg/vue_learn#9
Labels
C: VDialog VDialog T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke

Comments

@avenmore
Copy link

Environment

Vuetify Version: 2.0.19
Last working version: 2.0.11
Vue Version: 2.6.10
Browsers: Firefox 69.0
OS: Windows 10

Steps to reproduce

Show a dialog and press tab repeatedly.

Expected Behavior

When pressing tab, the focus cycles back to the first control of the dialog after the last control has been focused.

Actual Behavior

The focus goes out of the dialog, cycling through the page behind.

Reproduction Link

https://codepen.io/avenmore/pen/WNeBOZr

Other comments

Can be shown to be working in 2.0.11 in this pen (sorry, not mine, created to demo another problem). Setting "retain-focus" on the dialog has no effect.

@ghost ghost added the S: triage label Sep 28, 2019
@KaelWD KaelWD added C: VDialog VDialog T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke and removed S: triage labels Sep 29, 2019
@KaelWD
Copy link
Member

KaelWD commented Sep 29, 2019

fa0a893, #8459

@johnleider 3681cd5 was done specifically to allow for an override in cases like that

@mshmelev

This comment has been minimized.

@KaelWD

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VDialog VDialog T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke
Projects
None yet
3 participants