Bug 128882 – When applying style, attribute value matching should be case sensitive for SVG

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Benjamin Poulain]

When applying style, attribute value matching should be case sensitive for SVG

[Benjamin Poulain]

Created attachment 224314 [details]
Patch

[Andreas Kling]

Comment on attachment 224314 [details]
Patch

r=me, great catch!

[Benjamin Poulain]

Comment on attachment 224314 [details]
Patch

Clearing flags on attachment: 224314

Committed r164203: <http://trac.webkit.org/changeset/164203>

[Benjamin Poulain]

All reviewed patches have been landed.  Closing bug.

[Dirk Schulze]

I do not disagree that XML and SVG require case sensitiveness. So according to standards this is correct for now.

However, we try to harmonize SVG and HMTL where ever possible. Even the specification is slowly going into this direction. I am not sure if case sensitiveness for SVG is the right signal here.

[Benjamin Poulain]

(In reply to comment #5)
> However, we try to harmonize SVG and HMTL where ever possible. Even the specification is slowly going into this direction. I am not sure if case sensitiveness for SVG is the right signal here.

In the case of SelectorChecker, it is only doing case-insensitive value matching for a handful of legacy attributes. It is compatibility feature for old content.

This case-insensitive exception is nonsense in my opinion. It is impossible to understand for authors (the list is arbitrary and poorly documented) and those attributes are slower. We should aim toward removing this from HTML instead of adding it to SVG.

[Dirk Schulze]

(In reply to comment #6)
> (In reply to comment #5)
> > However, we try to harmonize SVG and HMTL where ever possible. Even the specification is slowly going into this direction. I am not sure if case sensitiveness for SVG is the right signal here.
> 
> In the case of SelectorChecker, it is only doing case-insensitive value matching for a handful of legacy attributes. It is compatibility feature for old content.
> 
> This case-insensitive exception is nonsense in my opinion. It is impossible to understand for authors (the list is arbitrary and poorly documented) and those attributes are slower. We should aim toward removing this from HTML instead of adding it to SVG.

Are there ambitions on WHATWG or somewhere else to drive this forward? Just to make it clear, I care less about case sensitiveness but more about consistency. Authors shouldn't be surprised because the same code that works in HTML doesn't work in SVG and vice versa. This is particularly bad for inline SVG embedded in HTML directly.

[Benjamin Poulain]

(In reply to comment #7)
> Are there ambitions on WHATWG or somewhere else to drive this forward? Just to make it clear, I care less about case sensitiveness but more about consistency. Authors shouldn't be surprised because the same code that works in HTML doesn't work in SVG and vice versa. This is particularly bad for inline SVG embedded in HTML directly.

There is a spec, it is followed by Firefox (and probably IE) -> we should match the spec.

You think there is matter to change the spec. It can go two ways: update SVG or fix HTML. I am in favor of fixing HTML. I do not participate in WHATWG so it is up to you if you want the change.

If you change the SVG spec (and Mozilla/Microsoft are on board), I am totally ok with "unfixing" this bug.