Remove unimplemented SVG color-profile and solidcolor #5147

ddbeck · 2021-05-20T12:05:22Z

What was wrong/why is this fix needed? (quick summary only)

Remove some pages and cross references to correspond to feature removals in BCD. See mdn/browser-compat-data#10515.

MDN URL of the main page changed

Chiefly but not exclusively:

https://developer.mozilla.org/docs/Web/API/SVGColorProfileElement
https://developer.mozilla.org/docs/Web/API/SVGSolidcolorElement
https://developer.mozilla.org/docs/Web/SVG/Element/color-profile
https://developer.mozilla.org/docs/Web/SVG/Element/solidcolor

Issue number (if there is an associated issue)

mdn/browser-compat-data#10515

Anything else that could help us review it

cc: @sideshowbarker

Content changes to correspond to BCD: mdn/browser-compat-data#10515

github-actions · 2021-05-20T12:07:42Z

Preview URLs

Flaws

Note! 5 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/SVG/SVG_2_support_in_Mozilla
Title: SVG 2 support in Mozilla
on GitHub
Flaw count: 22

macros:
- /en-US/docs/Web/API/SVGLocatable does not exist
- /en-US/docs/Web/API/SVGTransformable does not exist
- /en-US/docs/Web/API/SVGColor does not exist
- /en-US/docs/Web/API/SVGICCColor does not exist
- /en-US/docs/Web/API/SVGElementInstance does not exist
- and 17 more flaws omitted

URL: /en-US/docs/Web/SVG/Attribute/requiredFeatures
Title: requiredFeatures
on GitHub
Flaw count: 2

broken_links:
- Can't resolve /en-US/docs/DOM/DOMImplementation.hasFeature
- Can't resolve /files/3286/requiredFeatures.svg

URL: /en-US/docs/Web/SVG/Attribute/xlink:arcrole
Title: xlink:arcrole
on GitHub
Flaw count: 1

macros:
- /en-US/docs/Glossary/IRI does not exist

URL: /en-US/docs/Web/SVG/Content_type
Title: Content type
on GitHub
Flaw count: 7

macros:
- /en-US/docs/Web/API/SVGAnimatedAngle_objects does not exist
- /en-US/docs/Web/API/SVGColor does not exist
- /en-US/docs/Web/API/SVGPaint does not exist
broken_links:
- No need for the pathname in anchor links if it's the same page
- No need for the pathname in anchor links if it's the same page
- Is currently http:// but can become https://
- No need for the pathname in anchor links if it's the same page

URL: /en-US/docs/Web/API/Document_Object_Model
Title: Document Object Model (DOM)
on GitHub
Flaw count: 60

macros:
- /en-US/docs/Web/API/DOMConfiguration does not exist
- /en-US/docs/Web/API/DOMErrorHandler does not exist
- /en-US/docs/Web/API/DOMImplementationList does not exist
- /en-US/docs/Web/API/DOMImplementationRegistry does not exist
- /en-US/docs/Web/API/DOMImplementationSource does not exist
- and 55 more flaws omitted

External URLs

URL: /en-US/docs/Web/SVG/SVG_2_support_in_Mozilla
Title: SVG 2 support in Mozilla
on GitHub

https://bugzilla.mozilla.org/show_bug.cgi?id=1019326 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1174097 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1238882 (3 times)
https://bugzilla.mozilla.org/show_bug.cgi?id=1239100 (2 times)
https://bugzilla.mozilla.org/show_bug.cgi?id=1239128 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1239142 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1239147 (2 times)
https://bugzilla.mozilla.org/show_bug.cgi?id=1239218 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1240275 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1240357 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1245751 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1260032 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1265666 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1280425 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1302693 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1302705 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1303567 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1303569 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1325319 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1446617 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1446650 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=1451823 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=360148 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=528332 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=620002 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=719286 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=721920 (2 times)
https://bugzilla.mozilla.org/show_bug.cgi?id=733764 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=734079 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=778654 (3 times)
https://bugzilla.mozilla.org/show_bug.cgi?id=798843 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=828805 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=879659 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=921456 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=945187 (1 time)
https://bugzilla.mozilla.org/show_bug.cgi?id=999964 (1 time)
https://svgwg.org/svg2-draft/ (1 time)
https://www.w3.org/TR/SVG11/ (1 time)

URL: /en-US/docs/Web/SVG/Element
Title: SVG element reference
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Attribute/xlink:title
Title: xlink:title
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Attribute/name
Title: name
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Attribute/requiredFeatures
Title: requiredFeatures
on GitHub

https://www.w3.org/TR/SVG/feature.html (1 time)
https://www.w3.org/TR/SVG11/struct.html (1 time)

URL: /en-US/docs/Web/SVG/Attribute/xlink:arcrole
Title: xlink:arcrole
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Attribute/xlink:type
Title: xlink:type
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Attribute/xlink:href
Title: xlink:href
on GitHub

No new external URLs

URL: /en-US/docs/Web/SVG/Content_type
Title: Content type
on GitHub

https://example.com (1 time)
https://www.w3.org/TR/2001/REC-smil-animation-20010904/ (1 time)
https://www.w3.org/TR/2008/REC-CSS2-20080411/syndata.html (2 times)
https://www.w3.org/TR/2008/REC-CSS2-20080411/ui.html (2 times)
https://www.w3.org/TR/2008/REC-xml-20081126/ (1 time)
https://www.w3.org/TR/SVG/linking.html (1 time)
https://www.w3.org/TR/SVG/painting.html (1 time)
https://www.w3.org/TR/css3-color/ (2 times)

URL: /en-US/docs/Web/API/Document_Object_Model
Title: Document Object Model (DOM)
on GitHub

https://dom.spec.whatwg.org/ (1 time)

(this comment was updated 2021-05-20 12:49:49.966249)

chrisdavidmills

Nice work @ddbeck . A few bits I think need looking at, but nothing much.

Also, does @sideshowbarker want to look at this one?

files/en-us/web/svg/attribute/xlink_colon_arcrole/index.html

files/en-us/web/svg/attribute/xlink_colon_href/index.html

files/en-us/web/svg/attribute/xlink_colon_title/index.html

chrisdavidmills · 2021-05-20T12:21:12Z

files/en-us/web/svg/attribute/xlink_colon_type/index.html

@@ -10,7 +10,7 @@

 <p>The <strong><code>xlink:type</code></strong> attribute identifies the type of {{Glossary("XLink")}} being used. In SVG, only simple links are available.</p>

-<p>Twenty two elements are using this attribute: {{SVGElement("a")}}, {{SVGElement("altGlyph")}}, {{SVGElement("animate")}}, {{SVGElement("animateColor")}}, {{SVGElement("animateMotion")}}, {{SVGElement("animateTransform")}}, {{SVGElement("color-profile")}}, {{SVGElement("cursor")}}, {{SVGElement("feImage")}}, {{SVGElement("filter")}}, {{SVGElement("font-face-uri")}}, {{SVGElement("glyphRef")}}, {{SVGElement("image")}}, {{SVGElement("linearGradient")}}, {{SVGElement("mpath")}}, {{SVGElement("pattern")}}, {{SVGElement("radialGradient")}}, {{SVGElement("script")}}, {{SVGElement("set")}}, {{SVGElement("textPath")}}, {{SVGElement("tref")}}, and {{SVGElement("use")}}</p>
+<p>Twenty two elements are using this attribute: {{SVGElement("a")}}, {{SVGElement("altGlyph")}}, {{SVGElement("animate")}}, {{SVGElement("animateColor")}}, {{SVGElement("animateMotion")}}, {{SVGElement("animateTransform")}}, {{SVGElement("cursor")}}, {{SVGElement("feImage")}}, {{SVGElement("filter")}}, {{SVGElement("font-face-uri")}}, {{SVGElement("glyphRef")}}, {{SVGElement("image")}}, {{SVGElement("linearGradient")}}, {{SVGElement("mpath")}}, {{SVGElement("pattern")}}, {{SVGElement("radialGradient")}}, {{SVGElement("script")}}, {{SVGElement("set")}}, {{SVGElement("textPath")}}, {{SVGElement("tref")}}, and {{SVGElement("use")}}</p>


"Twenty-one"

This one got missed; update it to be consistent with the others, and I think we're good to go!

Fixed! Plus #5148

ddbeck · 2021-05-20T12:30:15Z

@chrisdavidmills Nice catch. Is the count particularly interesting? What do you think of the change I've just pushed?

chrisdavidmills

Just one comment. I think your approach here is fine, as long as all instances are consistent. I don't think the number is important or needed.

chrisdavidmills · 2021-05-20T12:32:59Z

files/en-us/web/svg/attribute/xlink_colon_type/index.html

@@ -10,7 +10,7 @@

 <p>The <strong><code>xlink:type</code></strong> attribute identifies the type of {{Glossary("XLink")}} being used. In SVG, only simple links are available.</p>

-<p>Twenty two elements are using this attribute: {{SVGElement("a")}}, {{SVGElement("altGlyph")}}, {{SVGElement("animate")}}, {{SVGElement("animateColor")}}, {{SVGElement("animateMotion")}}, {{SVGElement("animateTransform")}}, {{SVGElement("color-profile")}}, {{SVGElement("cursor")}}, {{SVGElement("feImage")}}, {{SVGElement("filter")}}, {{SVGElement("font-face-uri")}}, {{SVGElement("glyphRef")}}, {{SVGElement("image")}}, {{SVGElement("linearGradient")}}, {{SVGElement("mpath")}}, {{SVGElement("pattern")}}, {{SVGElement("radialGradient")}}, {{SVGElement("script")}}, {{SVGElement("set")}}, {{SVGElement("textPath")}}, {{SVGElement("tref")}}, and {{SVGElement("use")}}</p>
+<p>Twenty two elements are using this attribute: {{SVGElement("a")}}, {{SVGElement("altGlyph")}}, {{SVGElement("animate")}}, {{SVGElement("animateColor")}}, {{SVGElement("animateMotion")}}, {{SVGElement("animateTransform")}}, {{SVGElement("cursor")}}, {{SVGElement("feImage")}}, {{SVGElement("filter")}}, {{SVGElement("font-face-uri")}}, {{SVGElement("glyphRef")}}, {{SVGElement("image")}}, {{SVGElement("linearGradient")}}, {{SVGElement("mpath")}}, {{SVGElement("pattern")}}, {{SVGElement("radialGradient")}}, {{SVGElement("script")}}, {{SVGElement("set")}}, {{SVGElement("textPath")}}, {{SVGElement("tref")}}, and {{SVGElement("use")}}</p>


This one got missed; update it to be consistent with the others, and I think we're good to go!

chrisdavidmills

Perfect; nice one @ddbeck

sideshowbarker · 2021-05-21T04:56:24Z

Also, does @sideshowbarker want to look at this one?

I post-commit reviewed it, and it all looks great to me

Remove unimplemented SVG color-profile and solidcolor

27b7ddf

Content changes to correspond to BCD: mdn/browser-compat-data#10515

ddbeck requested a review from a team as a code owner May 20, 2021 12:05

ddbeck requested a review from a team May 20, 2021 12:05

ddbeck requested a review from a team as a code owner May 20, 2021 12:05

ddbeck requested review from chrisdavidmills and removed request for a team May 20, 2021 12:05

ddbeck mentioned this pull request May 20, 2021

Remove unimplemented SVG color-profile & solidcolor elements mdn/browser-compat-data#10515

Merged

chrisdavidmills requested changes May 20, 2021

View reviewed changes

Fix attribute usage miscounts

fac344f

chrisdavidmills requested changes May 20, 2021

View reviewed changes

Make xlink:type consistent

24b083d

ddbeck mentioned this pull request May 20, 2021

Drop SVG attribute counts #5148

Merged

chrisdavidmills approved these changes May 20, 2021

View reviewed changes

chrisdavidmills merged commit 6a1dddd into mdn:main May 20, 2021

ddbeck deleted the bcd-10515-removals branch May 20, 2021 13:20

github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove unimplemented SVG color-profile and solidcolor #5147

Remove unimplemented SVG color-profile and solidcolor #5147

ddbeck commented May 20, 2021

github-actions bot commented May 20, 2021 •

edited

Loading

chrisdavidmills left a comment

chrisdavidmills May 20, 2021

chrisdavidmills May 20, 2021 •

edited

Loading

ddbeck May 20, 2021

ddbeck commented May 20, 2021

chrisdavidmills left a comment

chrisdavidmills May 20, 2021 •

edited

Loading

chrisdavidmills left a comment

sideshowbarker commented May 21, 2021

Remove unimplemented SVG color-profile and solidcolor #5147

Remove unimplemented SVG color-profile and solidcolor #5147

Conversation

ddbeck commented May 20, 2021

github-actions bot commented May 20, 2021 • edited Loading

Preview URLs

Flaws

External URLs

chrisdavidmills left a comment

Choose a reason for hiding this comment

chrisdavidmills May 20, 2021

Choose a reason for hiding this comment

chrisdavidmills May 20, 2021 • edited Loading

Choose a reason for hiding this comment

ddbeck May 20, 2021

Choose a reason for hiding this comment

ddbeck commented May 20, 2021

chrisdavidmills left a comment

Choose a reason for hiding this comment

chrisdavidmills May 20, 2021 • edited Loading

Choose a reason for hiding this comment

chrisdavidmills left a comment

Choose a reason for hiding this comment

sideshowbarker commented May 21, 2021

github-actions bot commented May 20, 2021 •

edited

Loading

chrisdavidmills May 20, 2021 •

edited

Loading

chrisdavidmills May 20, 2021 •

edited

Loading