-
Notifications
You must be signed in to change notification settings - Fork 94
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
added describe to directional light and color variable example #447
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @yogitheboss, thank you for your contribution! The text content of the describe()
function is great.
We kindly request that you add the describe()
function to the liveSketch.js
examples rather than the .pde
files. You can find an explanation here. The describe()
function is only available in p5.js, not in Processing. The purpose of this change is to ensure that interactive examples are properly described for users who rely on screen readers.
Please don't hesitate to ask if you have any questions or concerns. Thank you again for your valuable contribution!
void setup() { | ||
|
||
describe('there are two dark spheres side by side ,the cursor works like a light source ,when mouse moves the direction of light changes and reveals some part of spheres.') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comma placement: a light source ,when
-> a light source, when
Ok @SableRaf , now i got it. |
Hey @yogitheboss, Just checking in on the Thanks! |
syncing with main
✅ Deploy Preview for processing-faf822 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thanks @yogitheboss for your contribution! ✨ It's a wonderful step in making the Processing website a bit more accessible. If you would like to make further alt-text contributions I'll make sure to get those merged a little faster next time 😉 |
this addresses issue #312