238954 - Support Command-. (Command-Period) keyboard shortcut for Stop on OS X

Status: RESOLVED FIXED

(Firefox :: Keyboard Navigation, defect)

Description

[Darkside]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.6) Gecko/20040206 Firefox/0.8
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.6) Gecko/20040206 Firefox/0.8

Command-Period is a stardard keyboard shortcut to Stop loading a page.  Internet
Explorer, Safari, and even Mozilla use it in addition to Escape (which is a
"hidden" shortcut, not listed in the menu).

Even though Mozilla doesn't list Command-Period as a keyboard shortcut in View >
Stop, it still works.  IE and Safari correctly list the shortcut in their menus.

Reproducible: Always
Steps to Reproduce:
1. Begin loading any webpage.
2. Press the Command and Period (.) keys.

Actual Results:  
The page continues to load.

Expected Results:  
The page should Stop loading, as if the Stop button was clicked or the View >
Stop menu item was selected.

Comment 1

[Jo Hermans]

*** Bug 250702 has been marked as a duplicate of this bug. ***

Comment 2

[Joseph R. Kiniry]

Confirmed; this bug still exists on Firefox 0.9.1 on OS X 10.3.4.  Escape stops
the load but neither Cmd-. and Cntl-. do anything at all.

Comment 3

[Asaf Romano (gone)]

no 1.0mac, wont-be-fixed for 1.0 (l10n freeze).

Comment 4

[Asaf Romano (gone)]

Attachment: v1

We should still support ESC on mac (but expose Accel+. in the UI).

Comment 5

[Asaf Romano (gone)]

mconnor, this one is pretty simple ; )

Comment 6

[Asaf Romano (gone)]

Comment on attachment 171440 [details] [diff] [review]
v1

needs update.

Comment 7

[Asaf Romano (gone)]

Attachment: v1.1

Comment 8

[Asaf Romano (gone)]

Checking in locales/en-US/chrome/browser/browser.dtd;
/cvsroot/mozilla/browser/locales/en-US/chrome/browser/browser.dtd,v  <-- 
browser.dtd
new revision: 1.12; previous revision: 1.11
done
Checking in base/content/browser-menubar.inc;
/cvsroot/mozilla/browser/base/content/browser-menubar.inc,v  <-- 
browser-menubar.inc
new revision: 1.45; previous revision: 1.44
done
Checking in base/content/browser-sets.inc;
/cvsroot/mozilla/browser/base/content/browser-sets.inc,v  <--  browser-sets.inc
new revision: 1.40; previous revision: 1.39
done