Skip to content

"Save All" button - Should it have a confirm action dialog? #2124

@dhaupin

Description

@dhaupin

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

  1. 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).
  2. Go home and work on those files some more via NppFTP using a different PC and session.
  3. 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.
  4. 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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions