processing / processing/p5.js-web-editor
Replace Popup Doesn’t Open When Find Popup is Active (via UI or Shortcut)
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.7k
- Forks
- 1.7k
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 8
Description
p5.js version
No response
What is your operating system?
None
Web browser and version
No response
Actual Behavior
When the Find popup is open (triggered via UI or the Ctrl + F shortcut), attempting to open the Replace popup (via UI or Ctrl + H shortcut) fails. The Find popup remains active, and the Replace popup does not appear until the user manually closes the Find popup.
This affects the user experience, as expected behavior would be to automatically switch between Find and Replace popups seamlessly.
https://github.com/user-attachments/assets/917261af-b21e-4cdc-9ab4-b33cb9b5f7b9
Expected Behavior
When the Replace popup is triggered, the Find popup should automatically close, and the Replace popup should be immediately visible.
Steps to reproduce
- Open the p5.js Web Editor.
- Press Ctrl + F or click the Find option → the Find popup appears.
- Now press Ctrl + H or click the Replace Text option.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the issue in the p5.js Web Editor by opening Find with Ctrl+F or its UI option, then triggering Replace with Ctrl+H or its UI option. Trace the Find and Replace popup components and their keyboard or UI handlers; done means triggering Replace closes Find and immediately displays Replace in both paths.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100