-
Notifications
You must be signed in to change notification settings - Fork 9
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
Enhanced 'MenuButton' component #2230
Comments
Update -- I see now that I need to actually place and render the menu myself. So the button really doesn't give me too much, other than a consistent way to render itself, and the little click handler to open the menu! |
The comment kind of says it all:
We have a bunch of 'preconfigured' buttons like this, across desktop and mobile, which are simply there to offer a consistent way to render and a click handler (e.g. FeedbackButton, OptionsButton, LogoutButton) |
Yes, my bad for not remembering that and wasting time. But the other three are really appropriate to be just simple triggers and can stay like that. There is no other code needed. Applications really need something like a MenuButton that takes arbitrary menu items and does the work for you of placement etc.! This is similar to our SplitButton project on Desktop, and could perhaps be folded in together in terms of adding something with consistent naming. |
Definitely agree that a component that does the work of creating and placing the menu for you would be much better - I'll see what can be done! |
Is this all specific to mobile? Note #1800 for earlier discussion from last spring about building a desktop |
Tom and I discussed this in-detail, and I gave him a few minor suggestions. Some big points we agreed on:
|
To answer your question ATM, we think it would be good to keep MenuButton API similar to what we think might be a SplitButton API, but no plans to make this cross-platform at this time (but eager to get your thoughts there) |
I was unable to figure out how to use the MenuButton component. It seems to have a very different API than what I would expect for a wrapped up control. In particular, it requires a Model be provided, rather just dropping in data for its items.
I understand that our Menu API is extremely mixed, but it would be nice if this control were a little simpler to drop in.
(Also could use a toolbox example of this)
The text was updated successfully, but these errors were encountered: