[webkit-reviews] review cancelled: [Bug 10794] [CSS1] Background-attachment: scroll should scroll within the containing block : [Attachment 20543] patch (updated)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 15 09:18:48 PDT 2008

Anatoli Papirovski <apapirovski at mac.com> has cancelled Anatoli Papirovski
<apapirovski at mac.com>'s request for review:
Bug 10794: [CSS1] Background-attachment: scroll should scroll within the
containing block

Attachment 20543: patch (updated)

------- Additional Comments from Anatoli Papirovski <apapirovski at mac.com>
Ok, hopefully this is the final patch. I've modified my test case to also test
background-origin for background-size on root element.

Also, while I was testing that I noticed that WebKit's implementation of
background-size: 50% auto;, that is percentage width and auto height, was
broken and so I just quickly fixed it... was there a reason why it was that
way, which I missed? (The formula that was there for calculation did not make
any sense what-so-ever.) There is no separate ticket for that and I didn't
think it was worth creating one for a change of one variable.

No test case was needed for that one, as there is one in fast/backgrounds/size
... it's the number 18.

More information about the webkit-reviews mailing list