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

feat(core): dialog focus trap #12831

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

khotcholava
Copy link
Contributor

closes #12768

Ensure the button does not lose focus after the dialog is closed, improving usability and navigation flow.

Uploading Screen Recording 2024-12-09 at 13.36.32.mov…

closes [#12768](#12768)

Ensure the button does not lose focus after the dialog is closed, improving usability and navigation flow.
Copy link

netlify bot commented Dec 9, 2024

Deploy Preview for fundamental-ngx ready!

Name Link
🔨 Latest commit 5992900
🔍 Latest deploy log https://app.netlify.com/sites/fundamental-ngx/deploys/676568c75a64ae00085661df
😎 Deploy Preview https://deploy-preview-12831--fundamental-ngx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

github-actions bot commented Dec 9, 2024

Visit the preview URL for this PR (updated for commit 5992900):

https://fundamental-ngx-gh--pr12831-12768-dialog-focus-i-wt09trbd.web.app

(expires Mon, 23 Dec 2024 12:58:02 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 41b993ee8e451bd7c6770b342ce142dc886eacff

@khotcholava khotcholava marked this pull request as ready for review December 18, 2024 15:15
@droshev droshev added this to the Sprint 141 - December 2024 milestone Dec 18, 2024
@droshev
Copy link
Contributor

droshev commented Dec 18, 2024

@dpavlenishvili @alexhristov14 can you review it? thanks

@droshev droshev self-assigned this Dec 20, 2024
dialogRef.afterClosed.pipe(
takeUntilDestroyed(this._destroyRef)
).subscribe(() => {
console.log(dialogConfig.focusTrapped);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove console.log

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

Successfully merging this pull request may close these issues.

[Accessibility: ACC-273.2] Dialog: Focus lost on closing the dialog
3 participants