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

slope is not deprecated #32449

Closed
estelle opened this issue Feb 26, 2024 · 4 comments · Fixed by #34767
Closed

slope is not deprecated #32449

estelle opened this issue Feb 26, 2024 · 4 comments · Fixed by #34767
Assignees
Labels
Content:SVG SVG docs

Comments

@estelle
Copy link
Member

estelle commented Feb 26, 2024

MDN URL

https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/slope

What specific section or headline is this issue about?

deprecated header

What information was incorrect, unhelpful, or incomplete?

the slope attribute is listed as useful for font-face, but never supported and deprecated.

slope is an attribute of the <feComponentTransfer> element's children: <feFuncR>, etc., and is supported, not deprecated, as far as I can tell.

What did you expect to see?

the slope page being up to date, and about it's use within <feComponentTransfer>, not font-face.

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

see https://developer.mozilla.org/en-US/docs/Web/SVG/Element/feComponentTransfer. It's also in the css filter effects pages.

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@github-actions github-actions bot added Content:SVG SVG docs needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. and removed Content:SVG SVG docs labels Feb 26, 2024
@bsmth bsmth removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Apr 5, 2024
@bsmth
Copy link
Member

bsmth commented Apr 5, 2024

Thanks, Estelle, just removing needs triage as it looks like the issue covers the requirements, so this needs content work, right?

@bsmth bsmth added the accepting PR Feel free to open a PR to resolve this issue label Apr 5, 2024
@Anurella
Copy link
Contributor

Anurella commented Apr 7, 2024

@bsmth
I think the steps to the page's modification might look like the below

  1. remove the deprecated message at the top
  2. rewrite what the slope attribute indicates and include this line (You can use this attribute with the following SVG element: )
  3. the current usage note is valid
  4. the specification link will be updated to this

Are you okay with the outline above? If yes, please assign me to the issue. Also, should we add an example section to it or just link to the element where it plays a role?

@bsmth
Copy link
Member

bsmth commented Jun 12, 2024

For anyone taking this: the duplicate/closed #24935 has more context ℹ️

@estelle
Copy link
Member Author

estelle commented Jun 18, 2024

The <feFuncR>, <feFuncB>, <feFuncG>, and <feFuncA> elements all support the slope attribute, and use it to set values for linear filters, such as brightness. This page says it's deprecated and only relevant to font. I arrived at the page from https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute#transfer_function_attributes. It is definitely not deprecated.

@estelle estelle self-assigned this Jul 10, 2024
@estelle estelle removed the accepting PR Feel free to open a PR to resolve this issue label Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:SVG SVG docs
4 participants