Bug 100278 - Add shortcut key 'n' to "Don't save" in the Save Document? dialog
Summary: Add shortcut key 'n' to "Don't save" in the Save Document? dialog
Status: RESOLVED DUPLICATE of bug 99326
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: All Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: needsUXEval
Depends on:
Blocks:
 
Reported: 2016-06-08 19:50 UTC by Dan Dascalescu
Modified: 2016-06-09 20:47 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
No shortcuts are visible for the buttons (29.72 KB, image/png)
2016-06-08 19:50 UTC, Dan Dascalescu
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Dascalescu 2016-06-08 19:50:27 UTC
Created attachment 125557 [details]
No shortcuts are visible for the buttons

1. Edit a document and make a change
2. Close the document (Ctrl+F4)

Calc/Writer et al. will open a dialog titled "Save Document", with three buttons: "Don't save", "Cancel" and "Save".

For consistency with other applications, 'N' should be the accelerator for "Don't save". Right now the shortcut for that is an awkward Alt+D, which isn't discoverable until the user presses Alt.
Comment 1 Joel Madero 2016-06-09 03:27:10 UTC
UX needs to decide this.
Comment 2 V Stuart Foote 2016-06-09 05:04:12 UTC
Sorry, that is by design and implemented for bug 92630

Those are not "shortcuts" those are "accelerators", refactoring was done to make their behavior compliant with GTK3 and only show the "localized" nemonic for the "accelerators" when the <Alt> key is pressed.

Assigning an actual shortcut is not appropriate.

Resolving duplicate of bug 99326, the final form there is still evolving as some prefer LibreOffice provide ability to optionally override the use of <alt> key and accelerator behavior.

*** This bug has been marked as a duplicate of bug 99326 ***
Comment 3 Dan Dascalescu 2016-06-09 07:39:19 UTC
@V Stuart: do you think users care more about the difference between "shortcut" and "accelerator" (I mentioned both terms in my report BTW), or about their muscle memory of pressing "N"?
Comment 4 Heiko Tietze 2016-06-09 07:54:38 UTC
I agree with Stuart that we should keep the first letter as accelerator. Would be interesting, by the way, where you see a missing consistency. 

Anyway, to improve the situation the text should be renamed to something like "Exit anyway" since "Don't save" doesn't tell the user how the operation is going on. Quite often this button is labeled "Discard changes". Good solution too.
Comment 5 V Stuart Foote 2016-06-09 13:32:05 UTC
(In reply to Dan Dascalescu from comment #3)
> @V Stuart: do you think users care more about the difference between
> "shortcut" and "accelerator" (I mentioned both terms in my report BTW), or
> about their muscle memory of pressing "N"?

What localization, what OS/DE?  

Why we in this BZ context care...

Accelerators normally are translated and receive appropriate mnemonic. Short-cuts are normally set elsewhere and are retained across localizations.  The "Discard/Close Without Saving/Don't Save" takes an accelerator. 

And actually the dialog discard any changes, and its mnemonic has been changed several times.

Most recently for bug 82017 -- "Close/Save confirmation dialog doesn't use CMD-D for don't save" for reasonable accommodation of the OS X side.  The .ui Glade GTK button object remains labeled "discard".

3.3 -> 3.6 label and accelerator was "~Discard"

4.0 -> 4.3 label and accelerator was "Close _Without Saving" (note move to .ui)

4.4 -> 5.2 label and accelerator is "_Don`t Save"

I would argue that the original "Discard" is the better *localizable* label
but there is no real reason to revert or change from commint
https://cgit.freedesktop.org/libreoffice/core/commit/?id=10abd657aacea884f520c97054625a94184c4aca

and certainly no reason to adopt an "N" entry.

Otherwise the only issue here is that the accelerator is not revealed--hence the dupe to bug 99326.
Comment 6 Adolfo Jayme Barrientos 2016-06-09 20:47:04 UTC
FWIW, I’ve already changed the mnemonic of this button once, because someone found it non-intuitive: https://cgit.freedesktop.org/libreoffice/core/commit/?id=10abd657aacea884f520c97054625a94184c4aca

I’m reluctant to change it again to something else.