Skip to content

SVG: add new page: requiredextensions attribute#41588

Merged
Josh-Cena merged 25 commits intomainfrom
230
Oct 21, 2025
Merged

SVG: add new page: requiredextensions attribute#41588
Josh-Cena merged 25 commits intomainfrom
230

Conversation

@estelle
Copy link
Copy Markdown
Member

@estelle estelle commented Oct 18, 2025

part of openwebdocs/project#230

the conditional processing attribute link is added in #41576

estelle and others added 5 commits October 18, 2025 02:26
Updated SVG attributes section to clarify categories and descriptions.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@estelle estelle requested review from a team as code owners October 18, 2025 14:50
@estelle estelle requested review from wbamberg and removed request for a team October 18, 2025 14:50
@github-actions github-actions bot added Content:WebAPI Web API docs Content:SVG SVG docs Content:Meta Content in the meta docs size/m [PR only] 51-500 LoC changed labels Oct 18, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 18, 2025

Preview URLs

Flaws (3)

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

URL: /en-US/docs/Web/SVG/Reference/Element/switch
Title: <switch>
Flaw count: 3

  • macros:
    • Macro produces link /en-US/docs/Web/SVG/Element which is a redirect
    • Macro produces link /en-US/docs/Web/SVG/Element which is a redirect
    • Macro produces link /en-US/docs/Web/SVG/Element which is a redirect

(comment last updated: 2025-10-21 16:07:44)

@github-actions github-actions bot removed the Content:Meta Content in the meta docs label Oct 18, 2025

## See also

- HTML [`requiredExtensions`](/en-US/docs/Web/HTML/Reference/Global_attributes/requiredExtensions) attribute
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume you mean SVG (i.e. just delete this line since it's already mentioned below)? This has never been defined in HTML

{{APIRef("SVG")}}

The **`requiredExtensions`** read-only property of the {{domxref("SVGGraphicsElement")}} interface reflects the {{SVGAttr("requiredExtensions")}} attribute of the given element.
The **`requiredExtensions`** read-only property of the {{domxref("SVGGraphicsElement")}} interface reflects the {{SVGAttr("requiredExtensions")}} SVG [conditional processing attribute](/en-US/docs/Web/SVG/Reference/Attribute#conditional_processing_attributes) of the given element.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally I don't thikn we should link the category since it's irrelevant for understanding the attribute itself, but only for understanding how attributes relate to each other. The previous paragraph is the standard verbiage for how it should be introduced

Comment thread files/en-us/web/api/svgswitchelement/index.md Outdated
<tr>
<th scope="row">Value</th>
<td>
space-separated <code><a href="">&lt;url&gt;</a> values</code>
Copy link
Copy Markdown
Member

@Josh-Cena Josh-Cena Oct 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use <list-of-urls> for SVG, linking to https://developer.mozilla.org/en-US/docs/Web/SVG/Guides/Content_type#list-of-ts. Remnant from SVG 1

</tbody>
</table>

For a description of the values, please refer to the [HTML `requiredExtensions`](/en-US/docs/Web/HTML/Reference/Global_attributes/requiredExtensions) attribute.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't exist

In addition to the aria, core, event, and presentation attributes, the `<switch>` element supports the conditional processing attributes:

- {{SVGAttr("requiredExtensions")}}
- :
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to fill this in

## Attributes

This element only includes global attributes.
In addition to the aria, core, event, and presentation attributes, the `<switch>` element supports the conditional processing attributes:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't seen this verbiage elsewhere. I believe the standard is to say nothing. If you want to add this, we need another PR to update the standard first, because the SVG docs are quite consistent.

Comment thread files/en-us/web/svg/reference/element/switch/index.md Outdated
Comment thread files/en-us/web/svg/reference/element/switch/index.md Outdated
@sideshowbarker sideshowbarker removed the request for review from a team October 20, 2025 00:27
Co-authored-by: Joshua Chen <sidachen2003@gmail.com>
Copy link
Copy Markdown
Member Author

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something happened to the PR.... need to fix something locally, but this wasn't what i i thought i pushed.... apologies.

Comment thread files/en-us/web/api/svggraphicselement/requiredextensions/index.md Outdated
Comment thread files/en-us/web/api/svggraphicselement/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/element/switch/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/element/switch/index.md Outdated
Copy link
Copy Markdown
Member Author

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I think I fixed everything locally. Sorry about the borked PR.

@estelle estelle requested a review from Josh-Cena October 20, 2025 11:53
Comment thread files/en-us/web/api/svggraphicselement/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/systemlanguage/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/requiredextensions/index.md Outdated
@Josh-Cena
Copy link
Copy Markdown
Member

Josh-Cena commented Oct 20, 2025

There might be some issue with your changes? I still see those before my review Actually just need the changes to be applied to the other pages as well!

Comment thread files/en-us/web/api/svganimationelement/requiredextensions/index.md Outdated
Comment thread files/en-us/web/svg/reference/attribute/systemlanguage/index.md Outdated
Copy link
Copy Markdown
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! Just further updated the See also per my previous review that we should not link indirectly related stuff

@Josh-Cena Josh-Cena merged commit 1db5597 into main Oct 21, 2025
13 checks passed
@Josh-Cena Josh-Cena deleted the 230 branch October 21, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:SVG SVG docs Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants