drop support for altGlyph

RESOLVED FIXED in Firefox 49

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: longsonr, Assigned: longsonr)

Tracking

(Blocks 1 bug, {dev-doc-complete, site-compat})

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
It's been dropped from SVG 2.
(Assignee)

Comment 1

3 years ago
Posted patch altglyph.txt (obsolete) — Splinter Review
Attachment #8735270 - Flags: review?(cam)
(Assignee)

Updated

3 years ago
Assignee: nobody → longsonr
Comment on attachment 8735270 [details] [diff] [review]
altglyph.txt

Review of attachment 8735270 [details] [diff] [review]:
-----------------------------------------------------------------

Very sorry for the delay, again.

The patch looks good.  Neither Chrome nor Edge support the <altGlyph> -> <tspan> fallback behaviour (which is what is being removed here and is most at risk of being used), though Safari still does.  I think it's safe to remove without telemetry.

Please make sure to get a DOM peer's review.

I wonder if, in the spirit of sending intent emails more consistently, we should send an Intent to Unimplement to dev-platform about this?

::: dom/system/gonk/tests/marionette/ril_jshint/jshint.js
@@ -416,5 @@
>  	status               : false,
>  	SVGAElement          : false,
> -	SVGAltGlyphDefElement: false,
> -	SVGAltGlyphElement   : false,
> -	SVGAltGlyphItemElement: false,

This is an imported library so I don't think we should change it.

::: layout/reftests/svg/nesting-invalid-01.js
@@ +1,4 @@
>  // Any copyright is dedicated to the Public Domain.
>  // http://creativecommons.org/publicdomain/zero/1.0/
>  
>  // These are all of the SVG 1.1 element names, except for container elements,

Can you update this comment to be more accurate?
Attachment #8735270 - Flags: review?(cam) → review+
Keywords: dev-doc-needed
Whiteboard: dev-doc-needed
Keywords: site-compat
Whiteboard: dev-doc-needed
(Assignee)

Comment 4

3 years ago
The webidl changes need a DOM peer review, even though they are only removal.
Attachment #8735270 - Attachment is obsolete: true
Attachment #8750123 - Flags: review?(peterv)
(Assignee)

Updated

3 years ago
Attachment #8750123 - Attachment is patch: true
Attachment #8750123 - Flags: review?(peterv) → review+

Comment 7

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9cf2c93a154e
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
(Assignee)

Comment 9

3 years ago
<altGlyphDef> and <altGlyphItem> have never been supported so it's a bit misleading to say that support has been dropped.
Thanks! Removed <altGlyphDef> and <altGlyphItem> from the doc.
You need to log in before you can comment on or make changes to this bug.