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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Modal, Frame(Nav), Sheet] Trapped virtual cursor inside dialogs #3931
Conversation
馃煛 This pull request modifies 7 files and might impact 5 other files. This is an average splash zone for a change, remember to tophat areas that could be affected. Details:All files potentially affected (total: 5)馃搫
|
17873a3
to
815560a
Compare
815560a
to
5cf863a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome Andrew, just one question about mobille vs desktop label / role
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, additions look good 馃
* Trapped virtual cursor inside dialogs * changelog * Add nav container label * Only applt attributes on small screens
WHY are these changes introduced?
Keyboard focus is trapped whoever virtual cursors are not
Current keyboard trapping
Before/After virtual cursor navigation
WHAT is this pull request doing?
How to 馃帺