-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Feature Request]: Expand all prop for the SideNavMenu #9604
Comments
@adamalston thanks for opening this issue. We are thinking we don't really want to support this, but also want to understand your reasons behind wanting this. Could you explain why this is needed? |
Essentially what the issue description says; we have a design doc with a button to expand all SideNavMenu items. I would like a streamlined way of invoking this functionality (via a prop) but if that's not an option, we can just use the hack I have in the description. |
@adamalston what I was trying to ask was why is this UX pattern useful or important to have? What problem is it solving? What research has been done for it? I think just having a design spec for it is not always a good enough reason, we have to consider the reasoning behind it. Can your design team provide more information? 🙂 |
Heard back from one of the designers working with my team:
Closing. |
I have to say, I just found myself in a similar situation to the original poster and needed to expand a sideNavMenu to notify the user of incoming chat messages in their list of chats and I was surprised to find there was no "expanded" property that could be used to control the open/close of the menu. I was further surprised that the original poster was asked to jump through hoops to justify their request. 🤷♂️ @adamalston Thank you very much for the hack. |
Summary
Add an expand all (
expandAll
?) prop for the SideNavMenu to expand all of the menus in the SideNav.Justification
A design document that my team was given has a button to toggle the expansion of all items in the SideNavMenu.
I can currently expand all menus in the SideNav using the following code but it is just a "hack" to show that it is possible:
Having a set property to accomplish this task would be much better.
Desired UX and success metrics
A user would click a button located in the SideNav which would toggle the expansion of all menus.
Required functionality
No response
Specific timeline issues / requests
No response
Available extra resources
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: