After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 119270 - [HIG] "Esc" should not close keybinding properties dialog
[HIG] "Esc" should not close keybinding properties dialog
Status: RESOLVED DUPLICATE of bug 101293
Product: gnome-control-center
Classification: Core
Component: [obsolete] Keybinding
git master
Other All
: High normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
AP3
: 116045 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-08-06 16:27 UTC by Shakti
Modified: 2007-01-06 18:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (1.65 KB, patch)
2003-08-06 16:29 UTC, Shakti
none Details | Review

Description Shakti 2003-08-06 16:27:50 UTC
As per HIG, in dialog boxes pressing "Esc" key should activate "cancel"
button but not "close" button. But in "Shortcuts" capplet
(gnome-keybindings-properties) when you press "Esc" it closes the dialog.

Steps to reproduce:
1) launch gnome-keybindings-properties
2) Press "Esc"

Expected behaviour: since the dialog box  contains close button it should
not close the window.

Actual behaviour: "Esc" closes the shortcuts window
Comment 1 Shakti 2003-08-06 16:29:50 UTC
Created attachment 18965 [details] [review]
proposed patch
Comment 2 Calum Benson 2003-08-07 16:13:55 UTC
Apologies for spam... marking as GNOMEVER2.3 so it appears on the official GNOME
bug list :)
Comment 3 Jonathan Blandford 2003-08-12 19:44:23 UTC
I'm happy to commit this, but I can't find where in the HIG this is
listed.  Can you tell me what section to look in?
Comment 4 Seth Nickell 2003-08-12 20:20:33 UTC
I can't really find this in the HIG either... However, since the other
preference pages don't do this and its not "default GTK behavior"
anyway, I would say we should apply this patch. If we want "Esc" to
close preference pages, property pages, and maybe all windows we
should make an HIG change to this effect and get it done everywhere.
Comment 5 Jody Goldberg 2003-08-12 20:23:43 UTC
Personally I'd rather go the other way and have all of the capplets
close on escape.
Comment 6 Shakti 2003-08-13 15:52:09 UTC
CC'ing Calum to get his opinion on this bug.

Thanks.
Comment 7 Calum Benson 2003-08-13 17:02:43 UTC
This is certainly one that's cropped up before, but when we were
writing the HIG, the majority view was that:

- Esc should only ever mean Cancel
- when present, the OK button should be the default (and therefore
activated by Enter)
- instant apply windows shouldn't have a default button, so that Enter
never activated a Close button (unless the user focused it first).

The argument was that it could be confusing if:

- pressing Esc sometimes left behind some changes (as would be the
case when it meant Close), and sometimes it didn't (as would be the
case when it meant Cancel)
- Esc was the shortcut for Close, but sometimes Esc was as well.

So much so that IIRC gtk was specifically changed so that Esc didn't
activate Close buttons any more.

The HIG vaguely alludes to this in the Default Buttons section of the
Windows chapter, but I agree it makes it far from obvious.
Comment 8 Calum Benson 2003-08-13 17:04:37 UTC
> - Esc was the shortcut for Close, but sometimes Esc was as well.
                                                  ^^^
                                                  Enter

Comment 9 Matthew Gatto 2004-01-06 00:24:43 UTC
This is a dupe of bug 116045 but all the action is here, so pasting
comments from that bug:

   Opened by merchan@phys.lsu.edu (Gregory Merchan):

   Escape should only leave a mode or cancel a dialog. The keybindings
   window is instant-apply (not cancellable) and not modal, so Escape
   should not close it.

   -- Additional Comments From Jody Goldberg 2003-06-26 18:31 --

   I'd actually prefer the converse.  All the capplets should close on
   Escape
Comment 10 Matthew Gatto 2004-01-06 00:26:48 UTC
*** Bug 116045 has been marked as a duplicate of this bug. ***
Comment 11 Calum Benson 2004-01-23 15:08:56 UTC
Updating status whiteboard to reflect a11y team's assessment of priority.
Comment 12 Glynn Foster 2004-03-01 04:52:14 UTC
Can we decide something about this please? Marking 'High' on account
of the patch.
Comment 13 Calum Benson 2004-10-21 16:50:52 UTC
Apologies for spam-- ensuring Sun a11y team are cc'ed on all current a11y bugs.
 Filter on "SUN A11Y SPAM" to ignore.
Comment 14 Sebastien Bacher 2005-01-02 18:24:08 UTC
so, what's the decision ? Should the capplets be closed or not by Esc ?
Comment 15 Calum Benson 2006-04-26 17:05:53 UTC
Apologies for spam... ensuring Sun a11y folks are cc'ed on all current accessibility bugs.
Comment 16 Thomas Wood 2007-01-06 18:50:17 UTC
This has been resolved as per the comments in bug 101293.

*** This bug has been marked as a duplicate of 101293 ***