Skip to content

fix: allow createRadio to remove element - #9154

Merged
ksen0 merged 5 commits into
processing:mainfrom
slash-init:fix/radio-remove
Sep 11, 2026
Merged

fix: allow createRadio to remove element#9154
ksen0 merged 5 commits into
processing:mainfrom
slash-init:fix/radio-remove

Conversation

@slash-init

@slash-init slash-init commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Resolves #9146

Changes:

  • Updated createRadio().remove() so calling remove() without an argument removes the radio element itself.
  • Added unit tests verifying removal from both the DOM and _elements.
  • Added coverage for removeElements() removing radio elements.

PR Checklist

  • npm run lint passes
  • [Inline reference] is included / updated
  • [Unit tests] are included / updated

@welcome

welcome Bot commented Sep 8, 2026

Copy link
Copy Markdown

🎉 Thanks for opening this pull request! For guidance on contributing, check out our contributor guidelines and other resources for contributors!
🤔 Please ensure that your PR links to an issue, which has been approved for work by a maintainer; otherwise, there might already be someone working on it, or still ongoing discussion about implementation. You are welcome to join the discussion in an Issue if you're not sure!
🌸 Once your PR is merged, be sure to add yourself to the list of contributors on the readme page !

Thank You!

@p5-bot

p5-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

Continuous Release

CDN link

Published Packages

Commit hash: 4e88268

Previous deployments

1c284ae


This is an automated message.

@ksen0
ksen0 merged commit 5e1b783 into processing:main Sep 11, 2026
4 checks passed
@slash-init
slash-init deleted the fix/radio-remove branch September 11, 2026 09:34
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.

[p5.js 2.0+ Bug Report]: createRadio().remove() does not remove the radio element

2 participants