320528 - [meta-ish] Cmd-. (period) / Escape / Cancel button parity

Status: RESOLVED WORKSFORME

(Camino Graveyard :: Accessibility, defect)

Description

[Herbert Schulz]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051216 Camino/1.0+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051216 Camino/1.0+

If you mistakenly press Cmd-D you can't dismiss the drop down sheet using Cmd-. from the keyboard.

Reproducible: Always

Steps to Reproduce:
1.With a page open, press Cmd-D (to get the Add Page to Bookmarks sheet to drop down).
2Cancelling via Cmd-. doesn't work, only ESC or clicking works.
3.

Actual Results:  
See above.

Expected Results:  
Cmd-. is a standard Mac command to Cancel and should, along with ESC, be bound to the Cancel button on that sheet.

I haven't checked other sheets.

Comment 1

[froodian (Ian Leue)]

Yeah, Cmd-period should consistently mapped to cancel for any sheet.  I think we should tweak the summary to include all sheets we don't do this for, since a seperate bug for each one would be messy.  Just mention if you want me to file them seperately and I'll fix.  Here's the list of menu items we should add this to:

Open File...
Save As...
Import Bookmarks...
Export Bookmarks...
Page Setup...
Clear History...
Add Page to Bookmarks...
Add Bookmark Folder...

Comment 2

[Herbert Schulz]

Howdy,

I agree. Put them all under this bug.

Good Luck,
Herb Schulz

Comment 3

[froodian (Ian Leue)]

Found another:

Open Location... (with toolbar hidden)

Comment 4

[Chris Lawson (gone)]

Why isn't this working by default? I thought cmd-. was automatically hooked up to cancel by the OS, like Esc is.

cl

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Wevah and I already have a few bugs on Cmd-period not working.

Comment 6

[Chris Lawson (gone)]

Found another one tonight. Cmd-. with a download selected in the Download Manager doesn't work. Esc does. Oops.

cl

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

(In reply to comment #6)
> Found another one tonight. Cmd-. with a download selected in the Download
> Manager doesn't work. Esc does. Oops.

Bug 298446

(In reply to comment #1)
> Open File...

Bug 168644

Somewhat-related bugs: bug 285531, bug 289432, bug 314306

There may be others already filed, but these are the ones I came up with after a very quick search.

Comment 8

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Putting this in the right place; sorry.

Comment 9

[froodian (Ian Leue)]

Found another - the sheet that comes up for Search the Web when the toolbar is minimized or text only (probably should have thought of it back in Comment 3)

Comment 10

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

We need to figure out how you assign two shortcuts to the same button to fix a lot of these, both for sheets/dialogues generated in code and those done as nibs.

Comment 11

[froodian (Ian Leue)]

The reason we don't get the cmd-./escape parody for free is because we have cmd-. as the keyboard shortcut for "Stop Loading Page" in the main menu.  Every place I could think of that uses escape also works for cmd-. if we change the menu item's shortcut.

I'm not sure what we should do about this.  Obviously cmd-. should stop a loading page, but maybe we should set that keyboard shortcut programmatically instead of in the nib?  I'm also not sure how Safari works around this...

Comment 12

[froodian (Ian Leue)]

Attachment: Patch v1

This is moderately evil, for a couple reasons:

1) It completely rewrites an OS method that gets called at every keystroke
2) There's gotta be a better way to do the bitwise arithmetic here, but I sure don't know what it is.  The tricky thing is that you can't just compare between event and menu item, since there's some device-specific information in the event's modifierFlags.  Additionally, non-function-key items with shift don't have the shift mask in the menu item - they're just stored as the capital version of the letter.  What I do works, I'm just not sure it's best.
3) It feels kinda sketchy to essentially ask MainController if we need to beep

On the other hand, it seems to fix all incarnations of this bug.

Comment 13

[Stuart Morgan]

Comment on attachment 259875 [details] [diff] [review]
Patch v1

So many flavors of no.

Comment 14

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Mass un-setting milestone per 1.6 roadmap.

Filter on RemoveRedonkulousBuglist to remove bugspam.

Developers: if you have a patch in hand for one of these bugs, you may pull the bug back to 1.6 *at that point*.

Comment 15

[Mark Mentovai]

Command-. works for me in all of these cases.  I'm on Leopard.  Was this an OS bug that got fixed?  Does the OS only steal command-. when View:Stop Loading Page is enabled?

Comment 16

[philippe (part-time)]

Just as with bug 168644, this appears to work correctly in all listed cases (10.5 and 10.6)

Comment 17

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Ditto bug 168644 comment 17 here: if it doesn't work on 10.4, we should just "[10.4]" the summary and leave it open until we drop 10.4 support.

Yay for the progress of time :)

Comment 18

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

It's unclear what this is tracking itself (as opposed to the meta part, where it's tracking all of just bug 168644 now).  Given that, and comment 15 and 17, I'm just going to WFM this now.  This has served its purpose.

> Yay for the progress of time :)

P.S. Hi, Ian!