Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect column title in arc flags illustration: "Large Arc Sweep Flag" #38495

Open
mootari opened this issue Mar 6, 2025 · 5 comments
Open
Labels
accepting PR Feel free to open a PR to resolve this issue Content:SVG SVG docs

Comments

@mootari
Copy link

mootari commented Mar 6, 2025

MDN URL

https://developer.mozilla.org/en-US/docs/Web/SVG/Tutorial/Paths

What specific section or headline is this issue about?

Illustration of large arc and sweep flag combinations (svgarcs_flags.png)

What information was incorrect, unhelpful, or incomplete?

The column label reads "Large Arc Sweep Flag".

https://github.com/mdn/content/blob/5f50e55f9c64273f69bd01ed16698fe32f1c20c2/files/en-us/web/svg/tutorial/paths/svgarcs_flags.png

What did you expect to see?

The column label should read "Large Arc Flag".

Do you have any supporting links, references, or citations?

The SVG 2 Paths spec: https://svgwg.org/svg2-draft/paths.html#PathElement

Do you have anything more you want to share?

The error can be found on a small number of other (non-MDN) sites and is likely a result of accidentally merging large-arc sweep-flag into large-arc-sweep-flag.

MDN metadata

Page report details
@mootari mootari added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 6, 2025
@github-actions github-actions bot added the Content:SVG SVG docs label Mar 6, 2025
@Josh-Cena Josh-Cena added accepting PR Feel free to open a PR to resolve this issue and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Mar 10, 2025
@Josh-Cena
Copy link
Member

Would be tricky to replace the diagram but yes the text should change

@mootari
Copy link
Author

mootari commented Mar 11, 2025

The text can be updated with a simple edit:

Image

It's kinda ironic though that an article about SVG is illustrated by low-resolution subpixel-antialiased PNGs. 😅

It would probably take a couple of hours at most to rebuild all of them as SVGs (for this particular page), but i have a feeling that maybe this shouldn't be done without taking a broader look at the other pages and defining some design guidelines first.

@Josh-Cena
Copy link
Member

I think the most ideal solution is to use an in-page live sample instead. If anyone's interested to contribute, there are ample examples in the SVG docs.

@mootari
Copy link
Author

mootari commented Mar 11, 2025

I'm not sure I'd agree, at least not as a replacement. Arguably the illustration can convey the effects better than live code could, it doesn't require loading an editor, and including additional markup for the legends would likely bloat the example code.

@Josh-Cena
Copy link
Member

It would look like this: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/transform-origin or this: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/stroke-width. Basically live-rendered page with its source code; no editors. Sure there's some extra markup, but the goal is not to look at the code anyway (the status quo has zero code), and certainly enables interested readers to quickly extract the source code they need, since each cell would be an independent graphic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepting PR Feel free to open a PR to resolve this issue Content:SVG SVG docs
Projects
None yet
Development

No branches or pull requests

2 participants