[Card] add containerStyle prop#4085
Conversation
| children: PropTypes.node, | ||
| /** | ||
| * Override the inline-styles of the children element. | ||
| */ |
There was a problem hiding this comment.
I'm wondering if containerStyle wouldn't be better. I would be using childrenStyle if we were cloning the children.
|
@oliviertassinari i did this changed |
oliviertassinari
changed the title
| * Can be used to render elements inside the Card. | ||
| */ | ||
| children: PropTypes.node, | ||
| /** |
There was a problem hiding this comment.
What about Override the inline-styles of the container.?
|
@RahavLussato can you squash? |
RahavLussato
force-pushed
the
patch-1
branch
2 times, most recently
from
46f0096 to
bae9094
Compare
|
@nathanmarks i did some clumsy squash, can you squash on merge ? |
|
@RahavLussato - |
|
@mbrookes @nathanmarks is it ok now ? |
|
@RahavLussato Nearly, you still need to reword the commit fro "[Card] add childrenStyle prop: to "[Card] add containerStyle prop" |
add containerStyle prop to have the ability to customize inner div styles. i needed this to give the div width of 100% then i can align the header to center and the body to left. Update Card.js Update Card.js Update Card.js rename childrenStyle to containerStyle Update Card.js [Card] add containerStyle prop
|
@mbrookes @nathanmarks done :) |
|
@RahavLussato Thanks for your contribution! 👍 |
5 participants
add childrenStyle prop to have the ability to customize the inner div styles.
i needed this to give the div width of 100% then i can align the header to center and the body to left.