superset icon indicating copy to clipboard operation
superset copied to clipboard

feat(accessibility): add tabbing to chart menu in dashboard

Open eschutho opened this issue 1 year ago • 15 comments

SUMMARY

This change adds the ability to tab through elements on a dashboard, the title, charts, and into the chart controls, and execute each of them with a keyboard.

https://github.com/apache/superset/assets/5186919/4dd70521-131a-438d-b370-ab7557e1176b

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • [ ] Has associated issue:
  • [ ] Required feature flags:
  • [ ] Changes UI
  • [ ] Includes DB Migration (follow approval process in SIP-59)
    • [ ] Migration is atomic, supports rollback & is backwards-compatible
    • [ ] Confirm DB migration upgrade and downgrade tested
    • [ ] Runtime estimates and downtime expectations provided
  • [ ] Introduces new feature or API
  • [ ] Removes existing feature or API

eschutho avatar Nov 29 '23 20:11 eschutho

Codecov Report

Attention: Patch coverage is 65.16854% with 31 lines in your changes are missing coverage. Please review.

Project coverage is 69.68%. Comparing base (1d571ec) to head (81efffb). Report is 3 commits behind head on master.

Files Patch % Lines
...dashboard/components/SliceHeaderControls/index.tsx 68.96% 17 Missing and 1 partial :warning:
...rontend/src/components/ErrorMessage/ErrorAlert.tsx 28.57% 10 Missing :warning:
...components/DashboardBuilder/DashboardContainer.tsx 25.00% 3 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #26138      +/-   ##
==========================================
- Coverage   69.69%   69.68%   -0.01%     
==========================================
  Files        1908     1908              
  Lines       74530    74612      +82     
  Branches     8309     8333      +24     
==========================================
+ Hits        51942    51993      +51     
- Misses      20535    20565      +30     
- Partials     2053     2054       +1     
Flag Coverage Δ
javascript 57.19% <65.16%> (+0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Dec 02 '23 02:12 codecov[bot]

/testenv up

eschutho avatar Dec 08 '23 19:12 eschutho

@eschutho Ephemeral environment spinning up at http://54.213.200.158:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

github-actions[bot] avatar Dec 08 '23 19:12 github-actions[bot]

When using keyboard navigation to tab through on a dashboard, the highlight on the chart title in dashboards doesn't seem to be appearing for me

yousoph avatar Dec 12 '23 00:12 yousoph

/testenv up

eschutho avatar Dec 15 '23 01:12 eschutho

@eschutho Ephemeral environment spinning up at http://54.202.14.64:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

github-actions[bot] avatar Dec 15 '23 01:12 github-actions[bot]

/testenv up

eschutho avatar Dec 21 '23 19:12 eschutho

@eschutho Ephemeral environment spinning up at http://52.35.108.27:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

github-actions[bot] avatar Dec 21 '23 19:12 github-actions[bot]

/testenv up

eschutho avatar Jan 19 '24 19:01 eschutho

@eschutho Ephemeral environment spinning up at http://35.87.72.46:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

github-actions[bot] avatar Jan 19 '24 19:01 github-actions[bot]

@eschutho Ephemeral environment spinning up at http://35.87.72.46:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

I think this looks good! Thanks @eschutho 🙏

kasiazjc avatar Jan 22 '24 12:01 kasiazjc

This changes the default behavior of the Edit Chart link, from opening in the same window to a new one. This might open an old can of worms — is it intentional?

rusackas avatar Feb 07 '24 19:02 rusackas

This changes the default behavior of the Edit Chart link, from opening in the same window to a new one. This might open an old can of worms — is it intentional?

No, not intentional. Thanks for flagging that.

eschutho avatar Feb 08 '24 00:02 eschutho

/testenv up

eschutho avatar Feb 12 '24 19:02 eschutho

@eschutho Ephemeral environment spinning up at http://35.90.66.154:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

github-actions[bot] avatar Feb 12 '24 19:02 github-actions[bot]

When I'm navigating the dropdown with arrows, the whole page scrolls with every click 🙁 And it looks like I can't navigate to Submenus

https://github.com/apache/superset/assets/15073128/9ec868f4-46c5-40bb-acdd-dab953baaeab

kgabryje avatar Feb 27 '24 16:02 kgabryje

When I'm navigating the dropdown with arrows, the whole page scrolls with every click 🙁 And it looks like I can't navigate to Submenus

Yeah, I didn't add the submenus yet. That's for a separate PR.

eschutho avatar Feb 27 '24 23:02 eschutho

Ephemeral environment shutdown and build artifacts deleted.

github-actions[bot] avatar Apr 08 '24 16:04 github-actions[bot]