Open Bug 768797 Opened 7 years ago Updated 4 years ago

Access key for "Don't save" button not working anymore in composition's "Save Message?" confirmation prompt on MAC

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

People

(Reporter: Usul, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: access, regression)

STR :

1. Cmd - N to compose a new message
2. close the compose window
3. warning mesage comes up

try to chosse don't save using either ctrl-d or cmd-d nothing happens.

I get this little warning in  the console
Which warning? Forgot to paste the text?
(In reply to :aceman from comment #1)
> Which warning? Forgot to paste the text?

I mean the popup windows asking you if you are sure that you want to save/nosave your draft.
Also, if you are on Trunk, did you notice that the shortcut for "don't save" changed to "n" in bug 759053? (Who may have done such a thing:-))

The shortcut keys should be shown on the buttons. Aren't they on Mac?
(In reply to :aceman from comment #3)
> Also, if you are on Trunk, did you notice that the shortcut for "don't save"
> changed to "n" in bug 759053? (Who may have done such a thing:-))

That works.

> The shortcut keys should be shown on the buttons. Aren't they on Mac?

No they aren't and the convention is to use the first letter in the button what ever the button says.
OK, so the "regression" is bug 759053.
If you do not agree with the reasoning there and still want to change the key back on Mac, let's CC some of the guys that agreed to the change. And notice it is in Toolkit.

And if the keys really are not shown on Mac, then I'd agree with you. How could anybody figure out the key is "n".

I'll see if the key can be made platform dependent.
Depends on: 759053
/mozilla/mobile/android/components/PromptService.js (View Hg log or Hg annotations)
    line 228 -- bTitle = PromptUtils.getLocaleString("DontSave");
/mozilla/mobile/xul/components/PromptService.js (View Hg log or Hg annotations)
    line 408 -- bTitle = PromptUtils.getLocaleString("DontSave");
/mozilla/toolkit/components/prompts/src/nsPrompter.js (View Hg log or Hg annotations)
    line 156 -- buttonLabel = PromptUtils.getLocalizedString("DontSave");

It is in JS files, but if they could be made preprocessed, it should be possible to have different keys for each platform.
It is a dupe by appearance but technically it is not. Bug 265277 was filed long ago and the cmd-D shortcut was implemented since, just not marked in that bug (unknown by which bug, but comment 0 here proves it has worked). But since bug 759053 this regressed again and was changed to Cmd-N.
Not sure how to mark the bugs now.
In 17.0.8 and 26.0a1 Daily neither n/cmd-n or d/cmd-d does anything (using OSX 10.7.5). It's definitely a problem, whichever bug best applies...
Summary: Keyboard shortcut for "don't save" in the warning dialog doesn't work anymore → Access key for "Don't save" button not working anymore in composition's "Save Message?" confirmation prompt on MAC
Ever since Mac OS Lion the default shortcut for Don't Save has been Command-[delete], and Command-d goes to the desktop if you are choosing a save location. Thunderbird seems to use Command-d in this way; if you are in the Save Message As dialogue box, then Command-d will change the path to the Desktop. Perhaps we should use Command-[delete] as the shortcut for Don't Save in the Mac version?
You need to log in before you can comment on or make changes to this bug.