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

Fixes #10116: Deprecate container-{device} vars #10125

Merged
merged 2 commits into from
Sep 1, 2013

Conversation

mdo
Copy link
Member

@mdo mdo commented Aug 24, 2013

Fixes #10116.
Swaps @container-{device} classes for @container-{size} classes. Also updates the customizer and grid to use the new variables (and fixes a missing @screen-*-min change).

/cc @cvrebert

@cvrebert
Copy link
Collaborator

I think you're gonna need to swap the defaulting in variables.less the other way around to preserve backward compatibility.

@cvrebert
Copy link
Collaborator

The Travis failure is unrelated and transient, so it's safe to ignore.

@cvrebert
Copy link
Collaborator

Looks good otherwise.

@ggam
Copy link
Contributor

ggam commented Aug 25, 2013

You may add a deprecation alert as in the other variables ;)

@mdo mdo merged commit f29ec45 into master Sep 1, 2013
@mdo mdo mentioned this pull request Sep 1, 2013
@cvrebert
Copy link
Collaborator

cvrebert commented Sep 1, 2013

@mdo Erm, the backward compatibility issue was never fixed...

@cvrebert cvrebert deleted the deprecate_screen_containers branch September 1, 2013 22:18
@cvrebert
Copy link
Collaborator

cvrebert commented Sep 1, 2013

Oh, didn't see 50bf4d4; nevermind!

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.

Match @container-* to the new @screen-* variable names
3 participants