[Card] Expand on Click#1806
[Card] Expand on Click#1806shaurya947 merged 4 commits intomui:masterfrom codeheroics:feat-cardExpander
Conversation
|
Just did a fixup on the first commit, if anyone tried the previous version I had forgotten to commit modifications and it didn't work, sorry about that. |
|
@hugo-agbonon I assume that the Otherwise, this looks fine to me from a high-level (yet to run the code though). @droberts84 @MichaelTaylor3d you guys have any feedback on this? |
src/card/card-expandable.jsx
Outdated
There was a problem hiding this comment.
Why do we have a onClick and not a onTouchTap?
|
I replaced the onClick events by onTouchTap events, and renamed @shaurya947 That's right, any direct child component of |
|
Sounds good @hugo-agbonon! Thanks! |
There was a problem hiding this comment.
@hugo-agbonon this will not work with React 0.14. Props can no longer be mutated.. see here: https://facebook.github.io/react/blog/2015/10/07/react-v0.14.html#breaking-changes
I'll try to fix this.
There was a problem hiding this comment.
@shaurya947 @cgestes already wrote a fix, see #1868 / #1872,
There was a problem hiding this comment.
My bad, missed that. Merged now.
This is an implementation proposal for #1354 : To be able to expand card by clicking on an element such as CardHeader, rather than only being able to do it using the CardExpandable button.
This adds a new
expanderprop for Card child components, which triggers the expand on click.I also updated the card documentation as to provide a demo of the proposed use.