fundamental-ngx icon indicating copy to clipboard operation
fundamental-ngx copied to clipboard

fix(core): action-sheet focus bug

Open shrvr opened this issue 2 years ago • 5 comments

Related Issue(s)

closes #8344

Description

  1. Added tab key handling.
  2. Solved focus-lost on close for Mobile view.

shrvr avatar Jul 26 '22 22:07 shrvr

Deploy Preview for fundamental-ngx ready!

Name Link
Latest commit 3d0f26368ce44b0d3f7e66526bbdaa0b90fcf336
Latest deploy log https://app.netlify.com/sites/fundamental-ngx/deploys/6303fa06ecda5400086f4680
Deploy Preview https://deploy-preview-8458--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 settings.

netlify[bot] avatar Jul 26 '22 22:07 netlify[bot]

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

https://fundamental-ngx-gh--pr8458-fix-8344-action-shee-ybuf49e7.web.app

(expires Thu, 25 Aug 2022 22:00:17 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

github-actions[bot] avatar Jul 26 '22 22:07 github-actions[bot]

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] avatar Aug 01 '22 00:08 github-actions[bot]

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] avatar Aug 07 '22 00:08 github-actions[bot]

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] avatar Aug 11 '22 00:08 github-actions[bot]

This pull request is stale because it has been open 2 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] avatar Aug 15 '22 00:08 github-actions[bot]

the first item in the list doesn't get focused after action sheet dialog opens, is it the intended behavior?

Yes, to give clear visual appearance in mobile mode. Users can focus first element using tab key if they use keyboard. Let me know your thoughts.

shrvr avatar Aug 18 '22 13:08 shrvr