At the URL given, in Opera 8.5, the character talking in the first frame has a beard, which isn't drawn in
Actually, both of them have missing stubble. It might be that it's supposed to have an alpha value, but
Opera is rendering it as 1.0 (black) instead of grey, and Safari is rendering it as 0.0 (white).
Created attachment 4336 [details]
The bug is when a fill attribute refers to a <pattern> element by fragment ID
Adding NeedsReduction keyword, the test case ideally should only be a couple lines.
Created attachment 4343 [details]
Reduced test case
There are 2 know bugs in
The first one is related with the use of viewBox. We really have to fix the
coordinates calculation when using this attribute as many SVGs use this.
The second bug (see the atteched reduced test case) is a coordinate generation
problem as well, but when using the pattern element. In this example, the first
(top-left) red <rect> should be rendered at (10.10) but it is rendered at
NOTE : the patternUnits="userSpaceOnUse" is important in this example. It means
that the coordinate system used for the pattern element (ie: for its x, y,
width and height values) is the same as the coordinate system of the black
Created attachment 4344 [details]
Created attachment 4345 [details]
MacDome, could you set the keyword to "HasReduction"
and the summary to something like "Pattern element coordinate problem" ?
Created attachment 4438 [details]
KRenderingPaintServerPatternQuartz was checking boundingBoxMode() of
KRenderingPaintServerPattern to shift the coordinates of the <pattern>.
That's not necessary as the x() and y() coordinates are already shifted thanks
to the baseVal() method.
By the way, the same code was already commented out a line 135
Created attachment 4439 [details]
Removes unnecessary comments
Comment on attachment 4438 [details]
We don't generally commit commented out code.
Comment on attachment 4439 [details]
Looks fine, I'll test again before committing.