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

[TreeView] Scroll not working when navigating via arrow keys #9961

Closed
2 tasks done
jnous-5 opened this issue Dec 5, 2022 · 4 comments · Fixed by #12226
Closed
2 tasks done

[TreeView] Scroll not working when navigating via arrow keys #9961

jnous-5 opened this issue Dec 5, 2022 · 4 comments · Fixed by #12226
Assignees
Labels
component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! design This is about UI or UX design, please involve a designer new feature New feature or request

Comments

@jnous-5
Copy link

jnous-5 commented Dec 5, 2022

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Steps to reproduce 🕹

Link to live example:
https://codesandbox.io/s/crazy-blackwell-v5mu7m?file=/demo.tsx

Steps:

  1. Navigate to each tree item via Arrow keys.
  2. Notice that as you move further down, the content doesn't scroll.

Current behavior 😯

The content doesn't scroll as the user navigates via Arrow keys.

Expected behavior 🤔

The content should scroll when navigating via Arrow keys, similar to the previous v4 behavior.

Context 🔦

This is an issue in terms of a11y as the user navigating via keyboard cannot see the entire list. I'd also want to add that this was mentioned here, although no further discussions were made.

Your environment 🌎

N/A. Can be replicated in the provided codesandbox.

@jnous-5 jnous-5 added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Dec 5, 2022
@zannager zannager added the component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! label Dec 5, 2022
@siriwatknp siriwatknp added new feature New feature or request and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Dec 19, 2022
@Max-ChenFei
Copy link

Hi, any update?

@joserodolfofreitas joserodolfofreitas transferred this issue from mui/material-ui Aug 7, 2023
@oliviertassinari oliviertassinari added design This is about UI or UX design, please involve a designer and removed design: ux labels Aug 18, 2023
@ilbrando
Copy link

The solution isn't as simple as removing preventScroll from this line in the code?

@amhci
Copy link

amhci commented Nov 1, 2023

This is a major accessibility issue for us. Visual, keyboard-only users cannot use the TreeView if it's overflowed and needs to scroll.

Copy link

⚠️ This issue has been closed.
If you have a similar problem, please open a new issue and provide details about your specific problem.
If you can provide additional information related to this topic that could help future readers, please feel free to leave a comment.

How did we do @jnous-5?
Your experience with our support team matters to us. If you have a moment, please share your thoughts through our brief survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! design This is about UI or UX design, please involve a designer new feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants