86880 2012-05-18 11:53:51 -0700 REGRESSION (r116645): "background-size: auto 100%; width:auto; height:100%;" doesn't display correctly 2012-05-21 14:34:22 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID http://peterlew.is/beta.html Regression P1 Normal --- 1 peter webkit-unassigned ap bdakin joethomas menard oldest_to_newest 627882 0 142755 peter 2012-05-18 11:53:51 -0700 Created attachment 142755 Safari vs Webkit nightly On http://iam.ptrl.ws/beta.html I'm using width:auto; height:100% to have a 2000x2000px displayed. In Webkit this doesn't display correctly and instead fills the whole browser window. 629348 1 ap 2012-05-21 12:02:12 -0700 The bug URL is 404 Not Found. Could you please attach your test here? 629357 2 peter 2012-05-21 12:09:01 -0700 Oh I am sorry, I changed my website's URL today! The URL of the page is: http://peterlew.is/beta.html Peter :) (In reply to comment #1) > The bug URL is 404 Not Found. Could you please attach your test here? 629419 3 ap 2012-05-21 13:13:55 -0700 Regressed in <http://trac.webkit.org/changeset/116645>. 629430 4 joethomas 2012-05-21 13:25:23 -0700 Here is the CSS from the regressed page background-size: auto 100%; width:auto; height:100%; background:url(pl_av_cp.jpg) center center no-repeat; Here background-size is defined before background shorthand. Since we added background-size to background-shorthand in http://trac.webkit.org/changeset/116645, if background-size is not part of background shorthand, we initialize it with the default value. So this can be solved either by adding background-size to background-shorthand or defining background-size after background. Peter, Can you please try this out? 629432 5 joethomas 2012-05-21 13:28:18 -0700 The same issue is mentioned in https://bugs.webkit.org/show_bug.cgi?id=27577#c2 629440 6 menard 2012-05-21 13:36:21 -0700 (In reply to comment #4) > Here is the CSS from the regressed page > > background-size: auto 100%; > width:auto; > height:100%; > background:url(pl_av_cp.jpg) center center no-repeat; > > Here background-size is defined before background shorthand. > > Since we added background-size to background-shorthand in http://trac.webkit.org/changeset/116645, if background-size is not part of background shorthand, we initialize it with the default value. So this can be solved either by adding background-size to background-shorthand or defining background-size after background. > > Peter, Can you please try this out? We render the same as FF and Opera now. So it's not really a regression. The behavior we had before was buggy. 629473 7 menard 2012-05-21 13:59:49 -0700 Previous behavior was wrong as we : - Didn't parse the background-size if it was part of the shorthand. We do now. - Didn't reset the background-size to initial if we don't find it in the shorthand, we do now. We now behave like the specification : http://www.w3.org/TR/css3-background/#the-background . If a value is missing in a CSS shorthand declaration we should set it to initial which we do now. 629510 8 peter 2012-05-21 14:34:22 -0700 Gosh you guys are quick to respond, thank you. I've made the adjustments as are recommended and it now renders correctly as can be seen in: http://peterlew.is/beta2.html So yes, the resolved/invalid status seems correct, thanks! :) 142755 2012-05-18 11:53:51 -0700 2012-05-18 11:53:51 -0700 Safari vs Webkit nightly url.txt text/plain 19 peter aHR0cDovL3B0cmwud3MvR2o0TA==