[ListItem] allowing for nestedindicator with righticonbutton presents

[gobadiah]

While working on nested lists, I have found a need for having the right nested button, plus another one (in my case it was a trash icon button for deleting that line and all the nested lines included).

I wonder if the state open / close shouldn't go into a prop, so that the user can decide whether or not to open the nested items regardless of the nested indicator ?

[oliviertassinari]

@gobadiah Could you add an example in the documentation, so we can check it out?

[gobadiah]

@oliviertassinari I added a right icon button in the nested list example, you can check it out.

[oliviertassinari]

Thanks. The render method of ths ListItem is starting to be far too big: 300 lines.
I think that it would be good to make it composable. I mean break it down into smaller bits, like what we plan to do with the TextField (#2555)
@alitaheri @newoga Though?

[alitaheri]

@oliviertassinari Yeah I agree with you. It's gonna be a bit hard though. But the idea is great 👍 👍 More composibility => easier customization + optimization + easier maintenance.

[newoga]

@oliviertassinari I'm sure you already know my answer 🎉 🙌 👍

[newoga]

That being said, maybe the best time to take this on (the breaking up of the render method into internal components) is when we clean up the directories and import structure.

[alitaheri]

I wonder if the state open / close shouldn't go into a prop, so that the user can decide whether or not to open the nested items regardless of the nested indicator

I can count 3-4 issues opened regarding this. I do think it's a great idea. But we have to give it some time. We plan on making every component stateless and standard so they can be wrapped inside state handling wrappers.

[mbrookes]

It looks like this PR got slightly derailed by a discussion about composability, but having tried the example, I'm not sure we should merge this (at least not in its current form). There's something about the resulting UI that doesn't feel right.

@callemall/material-ui ?

[oliviertassinari]

@mbrookes I agree.

[mbrookes]

@gobadiah - thanks for the contribution 👍, and sorry we couldn't use it this time.