Skip to content
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.

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

Feature: Auto-reload from local file #75

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

JeroenvdV
Copy link
Contributor

@JeroenvdV JeroenvdV commented Dec 6, 2023

Updates the 'Load from file' functionality to auto-reload the file contents if the file is changed on the local machine. It stops updating when other changes are made within the interface. Note that this isn't supported yet by all browsers, so the existing method is kept as a fallback.

Limitations:

  • Currently the auto-reload functionality is not optional and always on.
  • There is no single event that catches changes to the value of the textarea from both user actions and programmatically changing the value, so we rely on a dirty boolean which needs to be set manually whenever the value is updated in the program. This could be prone to future bugs.

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.

None yet

1 participant