-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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: add support for changing the surface mode in the menu #4378
Conversation
Hey @lenzi-erickson, thank you for your pull request 🤗. The documentation from this branch can be viewed here. |
Hey, |
|
Now I see your point. Change looks good to me. Can you add a test to this case? |
What are you wanting me to test, exactly? Since I'm just passing a prop through, and did not add any logic, I'm not sure what you are wanting me to test. |
I attempted to add some tests. If it's not what you are hoping for, I need some help getting it here, please. |
This is exactly what I was expecting. Thanks! |
@seb-zabielski What else do I need to do to get this merged? And any ETA on the next release? I would love to get this in. Thanks! |
Motivation
I prefer the "flat" design approach on my components, and would like to be able to use the
Menu
component in flat mode.Related issue
You cannot currently change the
mode
on theSurface
that is used in theMenu
component.Test plan
Pull up the menu example and pass
mode="flat"
to one of the menus to test.