-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[Breadcrumbs] Keep focus in the component after expanding #20489
[Breadcrumbs] Keep focus in the component after expanding #20489
Conversation
Details of bundle changes.Comparing: 89f6208...8025004 Details of page changes
|
5a02bfb
to
8025004
Compare
|
||
// The clicked element received the focus but gets removed from the DOM. | ||
// Let's keep the focus in the component after expanding. | ||
const focusable = event.currentTarget.parentNode.querySelector('a[href],button,[tabindex]'); |
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.
@oliviertassinari Thanks for adding the test case and changes. Next time, I will keep this in mind before making the PR. |
@shehryarshoukat96 Awesome :), if you see anything else that could be improved, feel free to mention it. |
@shehryarshoukat96 Thank you |
This PR successfully closes #20280. I have tested the code on IE-11 and Latest Chrome version.