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

Create a variable for retina display #13619

Closed
wants to merge 5 commits into from
Closed

Create a variable for retina display #13619

wants to merge 5 commits into from

Conversation

bassjobsen
Copy link
Contributor

@retina-displays can be used for extending Bootstrap

@cvrebert cvrebert added the css label May 17, 2014
@cvrebert
Copy link
Collaborator

Seems like a #13014-style approach would be better.

@bassjobsen bassjobsen mentioned this pull request May 17, 2014
@bassjobsen
Copy link
Contributor Author

@cvrebert you can have both we defining the following mixin:

    @retina-displays: ~"only screen and (-webkit-min-device-pixel-ratio: 2), only screen and (min--moz-device-pixel-ratio: 2), only screen and (-o-min-device-pixel-ratio: 2/1),  only screen and (min-device-pixel-ratio: 2),  only screen and (min-resolution: 192dpi),only screen and (min-resolution: 2dppx)";

// retina display
.screen-retina(@rules) {

  @media @retina-displays
     { @rules(); }
}

@bassjobsen bassjobsen closed this May 17, 2014
@bassjobsen bassjobsen reopened this May 17, 2014
@cvrebert
Copy link
Collaborator

@bassjobsen This currently fails to compile; see the Travis build output.

@bassjobsen
Copy link
Contributor Author

@cvrebert yes i see, i'm sorry for that. I see i commit three files (keyframes.less, progress-bars.less and mixins.less) by accidentally. And now i have to say sorry again because of i don't know howto fix this?

@mdo
Copy link
Member

mdo commented May 23, 2014

Passing on this for the time being, thanks though.

@mdo mdo closed this May 23, 2014
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.

3 participants