Skip to content

Conversation

NalinDalal
Copy link
Contributor

Resolves issue

Changes:
introduced yml file to auto close issue

checkout for reference

@ksen0
Copy link
Member

ksen0 commented Sep 2, 2025

Hi @NalinDalal , same feedback here as on the other PR, and also looks like the base is not correct. Please close this PR and open on that is based on main in the first place, and contains the yml with all the feedback included. As I mentioned before, it might be helpful if you do these one at a time (so, after the other one is corrected and merged) do avoid duplicate effort. Thank you!

@NalinDalal
Copy link
Contributor Author

ohh, okay makes sense

@ksen0 ksen0 merged commit c7aca89 into processing:main Sep 9, 2025
4 checks passed
ksen0 added a commit that referenced this pull request Oct 3, 2025
* solves docs issue 837 for p5.js 2.x

* Update version

* Merge pull request #812 from processing/feat/perf

Fully unmount sketches that go out of frame

* add updated editor images for intro tutorials

* update image paths and alt text for editor images in intro tutorials

* update screenshots with consistent browser theme

* fixed typo: frameCount is a variable not a function

* Update p5.js to v1.11.7

* Set iframe srcdoc attribute to blank to prevent iframe navigation

* submission for shapeSVG library

* rename file

* Update p5.js to v1.11.8

* Create Accessibility Issue template

* Update 4.accessibility.yml

* Update placeholders in 4.accessibility.yml

* Update 4.accessibility.yml to remove extra "steps to reproduce field"

Fixes an accidental inclusion of an unnecessary field

* Add issue-closing workflow to main branch

* remove extra things, another issue entertained so removed that

* pr branch update

---------

Co-authored-by: Dave Pagurek <[email protected]>
Co-authored-by: Dave Pagurek <[email protected]>
Co-authored-by: raclim <[email protected]>
Co-authored-by: Axel <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: limzykenneth <[email protected]>
Co-authored-by: Ms. Feng <[email protected]>
Co-authored-by: kit <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants