Skip to content

Auto-Refresh Button Doesn't Trigger Immediate Refresh #2511

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.

Sign up for GitHub

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

Jump to bottom
Closed
perminder-17 opened this issue Oct 18, 2023 · 4 comments · Fixed by #2522
Closed

Auto-Refresh Button Doesn't Trigger Immediate Refresh #2511

perminder-17 opened this issue Oct 18, 2023 · 4 comments · Fixed by #2522
Assignees
@perminder-17
Labels
Bug Error or unexpected behaviors Priority:Medium Moderately important, should be completed in a fair timeframe
Milestone
PATCH Release for...

Comments

@perminder-17
Copy link
Contributor

p5.js version

No response

What is your operating system?

None

Web browser and version

No response

Actual Behavior

Our current auto-refresh feature doesn't immediately refresh code output when the "auto-refresh" button is clicked. Auto-refresh, which should provide a means for developers to see the output corresponding to the code they've written, currently falls short, potentially leading to inconsistencies between the code and its output.

issuep5.webm

Expected Behavior

Upon clicking the "auto-refresh" button, the code output should refresh immediately, allowing users to see the impact of their code changes without requiring additional edits.

Steps to reproduce

Steps:

  1. Go to home page
    2.Click on auto-refresh.
@perminder-17 perminder-17 added the Bug Error or unexpected behaviors label Oct 18, 2023
@perminder-17
Copy link
Contributor Author

@lindapaiste Can i work on this?

@perminder-17
Copy link
Contributor Author

@raclim

@raclim
Copy link
Collaborator

raclim commented Oct 20, 2023

Thanks for catching this! Yes, please feel free to go ahead and work on it!

@raclim raclim added the Priority:Medium Moderately important, should be completed in a fair timeframe label Oct 20, 2023
@perminder-17
Copy link
Contributor Author

Thanks for catching this! Yes, please feel free to go ahead and work on it!

Thanks @raclim

perminder-17 pushed a commit to perminder-17/p5.js-web-editor that referenced this issue Oct 20, 2023
@SoniNisha
solved for issue processing#2511
a0ebe5c
This was referenced Oct 20, 2023
Merged
Closed
@raclim raclim added this to the MINOR Release milestone Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@perminder-17 perminder-17

Labels
Bug Error or unexpected behaviors Priority:Medium Moderately important, should be completed in a fair timeframe
Projects
None yet
Milestone
PATCH Release for 2.10.1
Development

Successfully merging a pull request may close this issue.

Trigger Immediate Refresh when ticking the Auto-Refresh checkbox perminder-17/p5.js-web-editor
2 participants
@raclim @perminder-17