Closed sriganeshres closed 4 months ago
🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already.
This Environment is provided by Release, learn more! To see the status of the Environment click on Environment Status below.
:wrench:Environment Status : https://app.release.com/public/Processing%20Foundation/env-b2eb0472ac
Thanks for opening this PR!
I feel like it looks good so far! One extra change I might make is that the aria-label
should be updated to indicate that this logo redirects to a user's sketch. I think it could help to try to do some screen reader testing here too to confirm this!
Due to the amount of time that's passed, I'm going to close this PR for now, but please feel free to reopen a new one with the requested changes, thanks!
Fixes #3083
https://github.com/processing/p5.js-web-editor/assets/120654479/2ae8c559-3a70-47cc-94b0-0d717b86e989
Changes:
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #3083