Skip to content

[CRUD] Fix accessibility of rows in Crud component #4825

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

Open
rkristelijn opened this issue Apr 2, 2025 · 0 comments
Open

[CRUD] Fix accessibility of rows in Crud component #4825

rkristelijn opened this issue Apr 2, 2025 · 0 comments
Labels
component: crud RFC Request For Comments scope: toolpad-core Abbreviated to "core" waiting for 👍 Waiting for upvotes

Comments

@rkristelijn
Copy link
Contributor

rkristelijn commented Apr 2, 2025

What's the problem?

Dear Toolpad team,

First of all thanks on this amazing work with MUI. It makes working with MUI so much simpler. I just discovered the Crud component and I'm amazed at the simplicity to set it up.

Right from the bat I see that the rowclick is used to navigate from the list-view to the detail-view. I want to share some concerns here;

  1. This blocks any inline edit that you may want to do in the list-view because the click navigates away
  2. This is very hard to make accessible, the keyboard navigation doesn't work and the screenreader is unable to make out there is a way to navigate from the rowclick to the detail view
  3. Multiselect is unusable as the first click navigates away

What are the requirements?

Instead i would like to suggest to make on of the columns of type hyperlink, to create a so-called drilldown. This is easy for screen readers to make out there is a link to the detail view, multiselect is enabled, and you can have drilldowns to other components as well if you want

What are our options?

No response

Proposed solution

Imagine you have two 'business components': Contact and Account. A Contact can belong to only one account.

The Contact Crud:

Id Contact Name Account
drilldown to contact 1 just some text drilldown to account 23
drilldown to contact 2 just some text drilldown to account 345

The Account Crud:

Id Account Name Primary Contact
drilldown to account 1 just some text drilldown to contact 456
drilldown to account 2 just some text drilldown to contact 1

Resources and benchmarks

No response

@rkristelijn rkristelijn added RFC Request For Comments status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Apr 2, 2025
@Janpot Janpot changed the title [RFC] Crud drilldown with hyperlink on id or title instead of navigate on rowClick [CRUD] Fix accessibility of rows in Crud component Apr 10, 2025
@prakhargupta1 prakhargupta1 added waiting for 👍 Waiting for upvotes and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: crud RFC Request For Comments scope: toolpad-core Abbreviated to "core" waiting for 👍 Waiting for upvotes
Projects
Status: Backlog
Development

No branches or pull requests

3 participants