feat(layouts): add @mixin for responsive support for rows #9115
Conversation
http://codepen.io/shortfuse/pen/AXyZLX (localhost test) |
ebf07ae
to
ac19662
Compare
@@ -326,3 +326,47 @@ | |||
} | |||
} | |||
} | |||
|
|||
// Only use when in row layout | |||
@mixin when-layout-row($element) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems like a whole lot of selectors (close 1.5kb), for something as simple as drawing a divider.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, I do have some PRs that would use this to fix issues with md-select, md-checkbox, md-input-container and md-datepicker. I had a tough time trying to squish it down though.
I realized I should expand this to |
Create @mixin that allows different layout breakpoints Fixes angular#9112
ac19662
to
b7c239c
Compare
Yeah, this is looking too bloated. Bumped radio-button from 3KB to 10KB. |
|
||
// Only use when in row layout | ||
@mixin when-layout-row($element, $parent: '') { | ||
@media (max-width: 599px) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have variables for those breakpoints
Mistakenly merged... just now reverted from |
Rejected as invalid PR |
Create @mixin that allows different layout breakpoints
Fixes #9112