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

Rename .flex-xs-* and .flex-items-xs-* to .flex-* and .flex-items-* #21218

Merged
merged 2 commits into from
Nov 27, 2016
Merged

Rename .flex-xs-* and .flex-items-xs-* to .flex-* and .flex-items-* #21218

merged 2 commits into from
Nov 27, 2016

Conversation

Starsam80
Copy link
Contributor

Same approach as #21217, but this time I changed the formatting of _flex.scss a little bit so it was more easier to add @if {...} @else {...}.

The only remaining xs classes after this PR are:

  • .col-xs
  • .col-xs-auto
  • .navbar-toggleable-xs
  • .hidden-xs-up
  • .hidden-xs-down

I plan on doing something about .col-xs(-auto) without affecting this (.col wouldn't match) but that takes a bit more work. .hidden-xs-(up|down) seem like they have a naming convention going on, so changing one may break it. I'll leave that for someone else to decide.

This commit just cleans up the formating of _flex.scss by changing
it so that it uses a single `@include media-breakpoint-up` instead
of multiple. It also aligns all of the CSS properties so it looks
a bit nicer.
@mdo
Copy link
Member

mdo commented Nov 27, 2016

Awesome, thanks <3

@mdo mdo merged commit 8d03109 into twbs:v4-dev Nov 27, 2016
@mdo mdo added this to the v4.0.0-alpha.6 milestone Nov 27, 2016
@Starsam80 Starsam80 deleted the remove-flex-xs branch November 27, 2016 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants