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

Support for height and width mixin? #20

Open
builtbylane opened this issue Aug 12, 2014 · 6 comments
Open

Support for height and width mixin? #20

builtbylane opened this issue Aug 12, 2014 · 6 comments
Milestone

Comments

@builtbylane
Copy link

No description provided.

@rafalbromirski
Copy link
Owner

Any suggestion about the syntax?

@include xyz($width, $height) { ... }

Is that what you meant?

@builtbylane
Copy link
Author

a shortcut for writing something like this:

@media screen and (max-height: 900px), screen and (max-width: 800px) { }

@rafalbromirski
Copy link
Owner

probably same for min-height and min-width. I was thinking about this:

  • max-size($width, $height) {...}
  • min-size($width, $height) {...}

It should work. Will add it to another milestone (v2.0)

@rafalbromirski rafalbromirski added this to the 2.0 milestone Aug 13, 2014
@builtbylane
Copy link
Author

Love that idea. Thanks for all your hard work.

@builtbylane
Copy link
Author

I can't imagine a use case, but you might want to consider an min-max-size() and max-min-size() as well.

@rafalbromirski
Copy link
Owner

good point. I also think that (device-)aspect-ratio mixin might be useful

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

No branches or pull requests

2 participants