Material-ui: [IconMenu] Should we add the component back into v1?

Created on 29 Oct 2017 · 5Comments · Source: mui-org/material-ui

It seems IconMenu was deleted in cf0934dda2ef83852ca094ac7250e2d562ce6156 for the @next branch.

Does @next includes a component similar to IconMenu to build a menu with an icon directly?

screen shot 2017-10-29 at 9 53 45 am

Maybe I missed something - the @next docs says it does:

screen shot 2017-10-29 at 9 49 33 am

But I can't find it other than the Doc Demo which doesn't abstract the logic to a component, so that we don't have to define event handlers, etc.

wontfix

Source

Sydney-o9

👍16

Most helpful comment

Bummer, yet another component we use in v0.19.x that is not ported to v1.0. We use this component in our mobile views.

skirunman on 30 Dec 2017

👍13

All 5 comments

But I can't find it other than the Doc Demo which doesn't abstract the logic to a component, so that we don't have to define event handlers, etc.

@Sydney-o9 You are right, we haven't built an IconMenu abstraction yet. The integration between the two is straightforward. You need to listen for the onClick and provide a anchorEl. I don't think that it's enough to justify an abstraction. Still, plus the accessibility, it might start to be interesting to abstract the logic. One last thing, I think that this potential IconMenu component should be stateless.

I have added the waiting for users upvotes tag. I'm closing the issue as I'm not sure people are looking for such abstraction. So please upvote this issue if you are. We will prioritize our effort based on the number of upvotes.