mwrock / RequestReduce

Instantly makes your .net website faster by reducing the number and size of requests with almost no effort.
www.requestreduce.org
Apache License 2.0
228 stars 49 forks source link

Background position bottom #173

Closed fredrikhaglund closed 12 years ago

fredrikhaglund commented 12 years ago

Version: 1.8.13

In the case you have a background image aligned to the bottom of a div-tag with dynamic height, RequestReduce fails to position the image at the bottom.

This pattern is often used to get a drop shadow or bottom in place for a box with dynamic height.

Steps:

.box-a { background: url(a.png) no-repeat 0 100% transparent; width: 250px; border: solid 1px black; }

.box-b { background: url(b.png) no-repeat 0 100% transparent; width: 250px; border: solid 1px black; }

Expected behavior: -Both images should align to the bottom of the sprite -The generated background-position should be "-0 bottom" respectively "-251px bottom"

Actual behavior: -Both images should align to the top of the sprite image -The generated background-position should be "-0 0" respectively "-251px 0"

mwrock commented 12 years ago

Thanks for reporting this Fredrik! If the containing Div have explicit heights then the vertical positioning was working but in the case where no explicit height is set as in your examples, RR resets the background image offset to 0 which results in the problem you observe. I now preserve the original offset if no containing height is provided. This will be available in the next release due in the next week.

fredrikhaglund commented 12 years ago

And thank you for creating and sharing RR!

Any chance you can make this patch available soon?

/Fredrik

28 apr 2012 kl. 15:57 skrev Matt Wrock reply@reply.github.com:

Thanks for reporting this Fredrik! If the containing Div have explicit heights then the vertical positioning was working but in the case where no explicit height is set as in your examples, RR resets the background image offset to 0 which results in the problem you observe. I now preserve the original offset if no containing height is provided. This will be available in the next release due in the next week.


Reply to this email directly or view it on GitHub: https://github.com/mwrock/RequestReduce/issues/173#issuecomment-5396919

mwrock commented 12 years ago

Hi Fredrik. Its available in the current release. It was deployed a few weeks ago. Let me know if you still have problems with it.