Skip to content
New issue

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

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

Already on GitHub? # to your account

FF97 SVG path d attribute and path() updates #11903

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Conversation

hamishwillee
Copy link
Collaborator

The SVG path is specified in the d attribute. FF97 allows you to use the attribute as a CSS property, specifying the new string with path() or none.

This updates the d and path() docs to note this, and provide basic examples.

Other docs work for this can be tracked in #11597

@hamishwillee hamishwillee requested a review from a team as a code owner January 11, 2022 03:13
@hamishwillee hamishwillee requested review from a team and rachelandrew and removed request for a team January 11, 2022 03:13
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs Content:SVG SVG docs labels Jan 11, 2022
@hamishwillee hamishwillee mentioned this pull request Jan 11, 2022
7 tasks
@github-actions
Copy link
Contributor

Preview URLs

Flaws

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

URL: /en-US/docs/Web/SVG/Attribute/Presentation
Title: SVG Presentation Attributes
on GitHub
Flaw count: 4

  • bad_bcd_links:
    • no explanation!
    • no explanation!
    • no explanation!
    • no explanation!

External URLs

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

No new external URLs


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

No new external URLs


URL: /en-US/docs/Web/CSS/path()
Title: path()
on GitHub

No new external URLs

| Specification | Status |
| ------------------------------------------------------------------------ | -------------------------------- |
| {{SpecName('CSS Shapes', '#funcdef-path', 'path()')}} | {{Spec2('CSS Shapes')}} |
{{Specifications}}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note, this is a little odd. Below you can see new and old spec links. The new spec link say module 2 but links to module 1

image

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmm that is a little odd - I'm going to merge this pr and see if I can do a bit of investigating on monday 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Rumyra FYI, I'm going to try move these into being BCD specs either way, so if you didn't solve this here, I will in the next iteration.

@Rumyra Rumyra merged commit 4733b32 into mdn:main Jan 14, 2022
@hamishwillee hamishwillee deleted the ff97dpath branch January 30, 2022 23:29
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Content:CSS Cascading Style Sheets docs Content:SVG SVG docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants