-
Notifications
You must be signed in to change notification settings - Fork 101
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
Add the describe() function to live examples #312
Comments
I can definitely take on this, but as it is quite easy, would you prefer to label it as "good first issue" or so? So that a newcomer (newer than me at least ^^') can tackle on this and get acquainted with VCS? |
Good idea! Let's see if someone wants to take that on :) |
Hi @SableRaf , i would like to work on this issue, only doubt is to which all examples do i have to add the describe function, can you point out these examples to me? |
Hi @himanshu007-creator. Ideally this should be added to all the live examples coded in p5. You could start with a few simple ones:
For example, in the On/Off live sketch, the describe() function could be added to the p5 code like so:
|
@SableRaf I would love to work on this issue. |
@yogitheboss That's the idea yes! Some might be easier than others. I have listed a few low hanging fruits in the message above. I'll assign this to you. Feel free to ask if you need any more input. |
I'd like to contribute to this issue. |
Let's see. @yogitheboss are you still working on this? |
Yes @SableRaf I am still working on this issue. |
For sure! No pressure 😃 I just asked because of @nikhilhvr's request. Good luck with your exams!! |
@SableRaf I see that the issue is still open. Is there anything more needed to be added to this issue? |
Hi @AaryaZ, and thank you for pointing that out. The issue is still open due to some pending changes. I had reviewed the PR from @yogitheboss (#447) and provided feedback. We're waiting for those updates. Please check the discussion there for more context. |
Hi @SableRaf , I'd like to contribute to this issue, if this issue is still open. |
Hi @umangutkarsh and thanks for your Pull Request. We have provided feedback in the replies. For future reference to others willing to contribute alt-text via the |
Since p5.js has a convenient function to make sketches accessible to screen readers, it would be a great idea to use it.
Here's an example of how to do it:
content/examples/...
and navigate to the example folder for which you want to add alt-text.liveSketch.js
files.setup
function add ans.describe()
function and pass it your alt-text as a string.For example:
Edit: see for reference 👇
July 30th 2024 edit: added step by step instructions
The text was updated successfully, but these errors were encountered: