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

img in horizontal card #34160

Merged
merged 2 commits into from
Jun 3, 2021
Merged

img in horizontal card #34160

merged 2 commits into from
Jun 3, 2021

Conversation

ffoodd
Copy link
Member

@ffoodd ffoodd commented Jun 1, 2021

Closes #34145

Simply adds img-fluid and rounded-start classes to horizontal card example to ensure copy-pasting from the docs matches the expectation.

Preview: https://deploy-preview-34159--twbs-bootstrap.netlify.app/docs/4.6/getting-started/theming/#theme-colors

Closes #34145 

Simply adds `img-fluid` and `rounded-start` classes to horizontal card example to ensure copy-pasting from the docs matches the expectation.
@ffoodd ffoodd changed the title docs(cards): img in horizontal card img in horizontal card Jun 1, 2021
@tungphan995

This comment has been minimized.

@sauravkr818
Copy link

hey @ffoodd we can simply add card-img class.
(it will make all the corners of the image rounded)

@ffoodd
Copy link
Member Author

ffoodd commented Jun 1, 2021

Which is probably not the expected result, in that case having rounded corners on the right looks weird IMHO.

@sauravkr818
Copy link

sauravkr818 commented Jun 1, 2021

Which is probably not the expected result, in that case having rounded corners on the right looks weird IMHO.

Yes you are right @ffoodd
But then please look in V4.5 HORIZONTAL CARDS section https://getbootstrap.com/docs/4.5/components/card/#horizontal
You might need to change the class in the <img> tag to img-fluid and rounded-left

Thanks man for considering the issue.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@mdo mdo merged commit 46045b4 into main Jun 3, 2021
@mdo mdo deleted the main-fod-docs-card branch June 3, 2021 06:08
marvin-hinkley-vortx pushed a commit to Vortx-Inc/bootstrap that referenced this pull request Aug 18, 2021
* docs(cards): img in horizontal card

Closes twbs#34145 

Simply adds `img-fluid` and `rounded-start` classes to horizontal card example to ensure copy-pasting from the docs matches the expectation.

* Update site/content/docs/5.0/components/card.md

Co-authored-by: Mark Otto <markd.otto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

V5 : [BUG] Horizontal Cards
4 participants