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

Card-group class looses its display:flex; when screen < 576px #30366

Closed
yoannlepetit opened this issue Mar 9, 2020 · 2 comments · Fixed by #30416
Closed

Card-group class looses its display:flex; when screen < 576px #30366

yoannlepetit opened this issue Mar 9, 2020 · 2 comments · Fixed by #30416

Comments

@yoannlepetit
Copy link

yoannlepetit commented Mar 9, 2020

Hi,

After upgrading from 4.3.1 to 4.4.1 I've noticed that .card-group class was loosing it attribute display: flex; when resizing screen under the 576px.

Here is the CSS class I get from Dev Tools on the documentation's bootstrap page (https://getbootstrap.com/docs/4.4/components/card/#card-groups):

@media (min-width: 576px) {
    .card-group {
        display:-ms-flexbox;
        display: flex;
        -ms-flex-flow: row wrap;
        flex-flow: row wrap
    }
   ...
}

In the documentation, it is said

Card groups use display: flex;
This may be confusing.

Operating system: macOs 10.15.3 (19D76)
Browsers:

  • Safari Version 13.0.5 (15608.5.11)
  • Firefox 73.0.1 (64 bits)
  • Brave Version 1.4.96 Chromium: 80.0.3987.132 (Official Build) (64-bit)
@yoannlepetit yoannlepetit changed the title Card-group class loose its display:flex; when screen < 576px Card-group class looses its display:flex; when screen < 576px Mar 9, 2020
@XhmikosR
Copy link
Member

XhmikosR commented Mar 9, 2020

/CC @twbs/css-review

@XhmikosR XhmikosR added the v4 label Mar 9, 2020
@ysds ysds added the docs label Mar 10, 2020
@mdo
Copy link
Member

mdo commented Mar 17, 2020

That's because they start off as stacked cards on mobile, where we don't need to add flex or anything else. We can modify the docs to be clearer here.

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

Successfully merging a pull request may close this issue.

4 participants