Skip to content
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

CardBody: provide hasNoPadding property to remove body padding #5606

Closed
KKoukiou opened this issue Mar 31, 2021 · 3 comments
Closed

CardBody: provide hasNoPadding property to remove body padding #5606

KKoukiou opened this issue Mar 31, 2021 · 3 comments

Comments

@KKoukiou
Copy link
Collaborator

KKoukiou commented Mar 31, 2021

In Cockpit we use everywhere card, but when these contain list we decided that it's too much wasted space to have the padding in the card body. So we remove it. It looks like this:

Screen Shot 2021-03-31 at 10 43 13

Would you be interested to expose a property to do that properly?

@tlabaj
Copy link
Contributor

tlabaj commented Mar 31, 2021

cc @mcoker @mceledonia

@mcoker
Copy link
Contributor

mcoker commented Mar 31, 2021

@KKoukiou you can omit <CardBody> to use a card without padding - would that work?

  <Card>
    This card has no padding
  </Card>

@KKoukiou
Copy link
Collaborator Author

KKoukiou commented Apr 1, 2021

@mcoker Sure :) I actually had noticed that but did not want to use it because the demos don't have such an example so I though it was not a valid usage. So since you agree that this is a valid usage, we don't need extra implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants