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

Reloading a file will mark all lines as changed in change history bar #12261

Closed
ghost opened this issue Sep 30, 2022 · 5 comments
Closed

Reloading a file will mark all lines as changed in change history bar #12261

ghost opened this issue Sep 30, 2022 · 5 comments
Assignees
Labels

Comments

@ghost
Copy link

ghost commented Sep 30, 2022

Title pretty much explain all there is to it.

Stock notepad++ 8.4.6 32bits (if that matter...). No plugins, besides standard ones.

@rdipardo
Copy link
Contributor

Scintilla's maintainer is aware, and posted this to the official mailing list:

An issue that has appeared with change markers is that SciTE's Revert command leads to the whole document getting a green change mark as revert just reloads each byte of the file. Since its useful to undo to before the revert, this point can't just be set as the start of change history to appear clear.

It may be possible to add a 'revert' state to the change history that could appear clear but its not obvious what the semantics of this state would be.

https://groups.google.com/g/scintilla-interest/c/9zNAgBG4Pd0/m/C46OGiz_CwAJ

There's nothing that Notepad++ can do but wait for a future Scintilla release.

@alankilborn
Copy link
Contributor

A possible workaround: instead of doing a Ctrl+r to Reload from disk, do a Ctrl+w (Close) followed by a Ctrl+Shift+t (Restore Recent Closed File).

And note that this doesn't help the situation where a file is externally modified and Notepad++ prompts to reload it, and you answer Yes to that. You'd have to follow that up with the workaround above.

@alankilborn
Copy link
Contributor

Additional discussion, including more about workarounds, is HERE.

@donho donho self-assigned this Nov 4, 2022
@donho donho added the scintilla dependent Can't be considered for N++ implementation unless/until Scintilla changes label Nov 26, 2022
@Skrell
Copy link

Skrell commented May 3, 2023

This feature also affects other plugins negatively... please add a way to COMPLETELY disable it not just the visible portion of it.

@alankilborn
Copy link
Contributor

This feature also affects other plugins negatively

Stating your case in the best way possible would include details on "negatively".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants