-
Notifications
You must be signed in to change notification settings - Fork 322
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
Background optimisation #150
Comments
@dhunkin not sure if we are on the same page but could you please check this fiddle: http://jsfiddle.net/S3GtV/ If you uncomment the last line in stylesheet then it clearly does not set background position to 0 50% but rather removes the image altogether. Thoughts? |
Sure, but please try to add this to styles: |
@dhunkin got it! It indeed does have incorrect position-y so apparently |
* Apparently background:0 does not set y position to 0 but 50%.
@dhunkin 1.1.3 is out with fix to this issue - thanks for all your help! |
* Apparently background:0 does not set y position to 0 but 50%.
|
background: none was optimised to background: 0 which is wrong in some cases.
The difference is in background-position. "background: none" sets all background properties to initial values, so "background-position" will be "0 0". "background: 0" however sets background-position to "0 50%". This ca be easily verified with Firebug or CSS editor built into Chrome.
The text was updated successfully, but these errors were encountered: