Hi,

Any update on this?

Is it OK to remove 'stroke="#0"' from:
  LayoutTests/svg/custom/invalid-fill-hex.svg
    <rect width="100" height="100" fill="#ff" stroke="#0"/>
?

If there are no objections, I'll upload a patch to remove it for review.

Yuzo


On Tue, Jun 15, 2010 at 5:52 PM, Yuzo Fujishima <yuzo@google.com> wrote:
Nikolas,

Thank you for your response.
I'll hold off committing https://bugs.webkit.org/show_bug.cgi?id=24742 until
you make a decision. Please let me know when you make one.

Yuzo

On Tue, Jun 15, 2010 at 3:42 PM, Nikolas Zimmermann <zimmermann@physik.rwth-aachen.de> wrote:

Am 15.06.2010 um 08:28 schrieb Cameron McCormack:


Nikolas Zimmermann:
We had 'none' for years, until compatibility bugs were raised, as
Opera uses black as default stroke, if there is any error.

Doesn’t seem to be the case any more, at least for Opera 10.53:

http://livedom.validator.nu/?%3C!DOCTYPE%20html%3E%0D%0A%3Csvg%20width%3D'100'%20height%3D'100'%3E%3Ccircle%20cx%3D'50'%20cy%3D'50'%20r%3D'30'%20fill%3D'yellow'%20stroke%3D'xxx'%20stroke-width%3D'4'%2F%3E%3C%2Fsvg%3E

Note the stroke="xxx".  Firefox nightlies behave the same.

That would be good news, I've been against the switch all time, we should reiterate that specific behaviour. Turning off that default stroke behaviour and seeing which layout tests break, and whether we now consider this a progression :-)
I'll have a look at it this week.

Cheers,
Niko


_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev