-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Description
Description of the Issue
The save all button has no confirm, so a mis-click or mis-key will init the save-frenzy. This has undesirable or even dangerous effects if you are in a session with old files, possibly production, possibly in NppFTP....which is exactly what happened recently.
Steps to Reproduce the Issue
- Open a session at work, use NppFTP if you like. This session may have lots of old work in tabs. Work on a critical file(s).
- Go home and work on those files some more via NppFTP using a different PC and session.
- Come back to work, continue on 1 file, but accidently hit save all, watch as NppFTP overwrites all the old files open in the session, effectively wiping out the work you did at home.
- Erm....go back home to get the files? I dunno it's just annoying from here on out :)
Expected Behavior
Exactly what Np++ did. Although some sort of "are you sure you want to save everything in this session" dialog would be very much appreciated.
Actual Behavior
Np++ did exactly what is expected and normal behavior, it saved everything. This can have undesirable effects if you aren't expecting it.
Workaround
By using the "customize toolbar" plugin one can remove the icon for save all, reducing mis-clicks. Changing the shortcut to something more complex helps too. But for people who are using stock Np++ and fire off a save all by mistake at the wrong time, a confirm action dialog could save their butts.