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

fix: auto evaluate doesn't work with undo. #2106

Merged
merged 2 commits into from
Feb 16, 2025
Merged

Conversation

paxcut
Copy link
Contributor

@paxcut paxcut commented Feb 6, 2025

The pattern editor was resetting the flag that marked text changes on every frame, but resolving a text change may take more than one frame, so instead I created a function that allows the resetting of the text changed boolean from view pattern editor.

Currently, when a pattern is evaluated the pattern editor will lose focus regardless of how evaluation was triggered. This is specially annoying when using f5 or auto-evaluate. Now whenever a pattern is evaluated, focus is given to the pattern editor when evaluation ends.

JumptoLine didn't work for empty lines.

The pattern editor was resetting the flag that marked text changes on every frame, but resolving a text change may take more than one frame, so instead I created a function that allows the resetting of the text changed boolean from view pattern editor.

Currently, when a pattern is evaluated the pattern editor will lose focus regardless of how evaluation was triggered. This is specially annoying when using f5 or auto-evaluate. Now whenever a pattern is evaluated, focus is given to the pattern editor when evaluation ends.
@WerWolv WerWolv enabled auto-merge (squash) February 16, 2025 11:54
@WerWolv WerWolv merged commit 007e04b into WerWolv:master Feb 16, 2025
20 checks passed
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.

2 participants