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

Missing an attribute: specularConstant #36238

Closed
LibraSun opened this issue Oct 7, 2024 · 4 comments
Closed

Missing an attribute: specularConstant #36238

LibraSun opened this issue Oct 7, 2024 · 4 comments
Labels
closed: question A user misinterpretation or support question. No action required. Content:SVG SVG docs

Comments

@LibraSun
Copy link

LibraSun commented Oct 7, 2024

MDN URL

https://developer.mozilla.org/en-US/docs/Web/SVG/Element/feSpotLight

What specific section or headline is this issue about?

ATTRIBUTES

What information was incorrect, unhelpful, or incomplete?

No mention is made of the specularConstant attribute, which represents the ks value in the Phong shading model. Should be included, as it is part of the feSpecularLighting spec.

What did you expect to see?

specularConstant listed just before specularExponent

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

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

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@LibraSun LibraSun added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Oct 7, 2024
@github-actions github-actions bot added the Content:SVG SVG docs label Oct 7, 2024
@Josh-Cena
Copy link
Member

I don't see specularConstant here: https://drafts.fxtf.org/filter-effects/#feSpotLightElement do you have proof that specularConstant is supported for feSpotLightElement?

@Josh-Cena Josh-Cena added needs info Needs more information to review or act on. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Oct 7, 2024
@LibraSun
Copy link
Author

LibraSun commented Oct 7, 2024 via email

@Josh-Cena
Copy link
Member

Please check the following pages:

They are consistent about one thing: specularConstant is an attribute of feSpecularLighting, not an attribute of feSpotLight.

It is mentioned on the page you linked me to:

I can't see the image on GitHub, but all mentions of the specularConstant attribute are from the feSpecularLighting element, not feSpotLight.

Gemini AI engine refers to it when asked.

Never ever trust AI responses about technical facts.

The page I linked you to on MDN defines it.

That page explicitly says it's only supported for feSpecularLighting.

Please make sure that you are not mixing up feSpecularLighting and feSpotLight. I'm closing this for now, but if there's actually proof that browsers support specularConstant on the feSpotLight element, via a code example that can be directly viewed in browser, I'm happy to reopen.

@Josh-Cena Josh-Cena closed this as not planned Won't fix, can't repro, duplicate, stale Oct 7, 2024
@Josh-Cena Josh-Cena added closed: question A user misinterpretation or support question. No action required. and removed needs info Needs more information to review or act on. labels Oct 7, 2024
@LibraSun
Copy link
Author

LibraSun commented Oct 7, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed: question A user misinterpretation or support question. No action required. Content:SVG SVG docs
Projects
None yet
Development

No branches or pull requests

2 participants