-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
fix: Improving form widget keyboard accessibility #18991
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@jsartisan Noticing some inconsistencies here: |
@jsartisan Also, focus is seen on the widgets when they are set to |
Tested and verified the fixes, keyboard accessibility between widgets and wrapping within modals. |
app/client/src/components/designSystems/appsmith/PositionedContainer.tsx
Outdated
Show resolved
Hide resolved
/ok-to-test sha=2c7b7d1 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3805296503. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/IDE/Command_Click_Navigation_spec.js |
This PR allows tabbing between widgets according to their position.
Also, it allows trapping focus in the modal widget
Screen.Recording.2022-12-22.at.4.02.56.PM.mov
Fixes #7757
Type of change
How Has This Been Tested?