This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 24424 - Add parser support for feDropShadow
Summary: Add parser support for feDropShadow
Status: RESOLVED FIXED
Alias: None
Product: WHATWG
Classification: Unclassified
Component: HTML (show other bugs)
Version: unspecified
Hardware: All All
: P2 normal
Target Milestone: Unsorted
Assignee: Ian 'Hixie' Hickson
QA Contact: contributor
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 24428
  Show dependency treegraph
 
Reported: 2014-01-28 10:24 UTC by Henri Sivonen
Modified: 2014-01-30 02:38 UTC (History)
6 users (show)

See Also:


Attachments

Description Henri Sivonen 2014-01-28 10:24:26 UTC
https://dvcs.w3.org/hg/FXTF/raw-file/e1ae5deb9fa8/filters/index.html specifies <feDropShadow> in violation of the expectation that no new camel-case  names would be added to SVG.

Since camel-casing this is consistent with the existing filter effects, please add a mapping from fedropshadow to feDropShadow  to the parsing spec.
Comment 1 Robert Longson 2014-01-28 10:59:31 UTC
That specification also adds feCustom, feCustomSource and feCustomParam. Presumably these would need similar treatment.
Comment 2 Dirk Schulze 2014-01-28 11:54:08 UTC
(In reply to Henri Sivonen from comment #0)
> https://dvcs.w3.org/hg/FXTF/raw-file/e1ae5deb9fa8/filters/index.html
> specifies <feDropShadow> in violation of the expectation that no new
> camel-case  names would be added to SVG.

Define "violation"?

> 
> Since camel-casing this is consistent with the existing filter effects,
> please add a mapping from fedropshadow to feDropShadow  to the parsing spec.

Yes, that would be awesome. Thanks.
Comment 3 Dirk Schulze 2014-01-28 11:54:34 UTC
(In reply to Robert Longson from comment #1)
> That specification also adds feCustom, feCustomSource and feCustomParam.
> Presumably these would need similar treatment.

Hold back with these elements for now please.
Comment 4 Simon Pieters 2014-01-28 13:11:29 UTC
Would it be better to use all-lowercase for new SVG element and attribute names even if there are related features with camel-case already?

If yes, is it possible to change these to lowercase now?
Comment 5 Cameron McCormack 2014-01-28 16:57:50 UTC
(In reply to Simon Pieters from comment #4)
> Would it be better to use all-lowercase for new SVG element and attribute
> names even if there are related features with camel-case already?

That's the funadmental problem with saying all new elements/attribute will be lowercase, of course.  Sounds good in theory, but completely violates author expectations for people who are familiar with the existing elements/attributes.

I still hold out hope that we can move to a world where we support everything lowercased per http://dev.w3.org/SVG/proposals/improving-svg-dom/.
Comment 6 Ian 'Hixie' Hickson 2014-01-28 18:50:22 UTC
So what's the story here? Should I be changing the parser? Is someone else taking ownership of the issue while we figure out what needs to happen?
Comment 7 Ian 'Hixie' Hickson 2014-01-28 18:52:09 UTC
Note that I don't think there's anything wrong with adding new camelcase names. It's not like it saves the browsers much work: adding a new mapping is cheap compared to adding support for the actual feature, and there's not much point using the feature before browsers support it.
Comment 8 Robert Longson 2014-01-28 19:22:34 UTC
Chrome already supports feDropShadow and there's a Firefox bug almost ready to land that would implement it too, camel case and all.
Comment 9 Henri Sivonen 2014-01-29 06:20:19 UTC
(In reply to Ian 'Hixie' Hickson from comment #6)
> Should I be changing the parser?

Yes. Gecko implementation is tracked in https://bugzilla.mozilla.org/show_bug.cgi?id=964200

(In reply to Robert Longson from comment #8)
> Chrome already supports feDropShadow

Shame on them for not filing this bug!
Comment 10 contributor 2014-01-30 02:38:30 UTC
Checked in as WHATWG revision r8443.
Check-in comment: Update the parser to support new SVG camelCase name 'feDropShadow'
http://html5.org/tools/web-apps-tracker?from=8442&to=8443