-
-
Notifications
You must be signed in to change notification settings - Fork 215
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
Two dimensional fix #225
Two dimensional fix #225
Conversation
I've added another test to test the max-media-query & two dimensional combo here. |
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.
is there still something I have to do? |
974c8a8
to
5325c20
Compare
Nope, thanks. Just edited your commit message since you only referred to the Sass variable. I just need an approval from @XhmikosR to get this merged (Don't squash). |
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.
LGTM AFAICT
* Two dimensional fix - twbs/rfs#225
* Two dimensional fix - twbs/rfs#225
* Two dimensional fix - twbs/rfs#225
Fixing bug where two-dimensional mode wouldn't correctly scale the desired values. tests are adjusted accrodingly. Yet there are still tests to be written.
Fixes #223
Not sure what happend to #224, but when I pushed to the branch, the branch closed.