Contacts sidebar: Implement Ctrl+I keyboard shortcut to edit the properties of selected AND focused contact (ux-consistency with TB address book and MAC default shortcut for "Properties")

RESOLVED FIXED

Status

Thunderbird
Message Compose Window
--
enhancement
RESOLVED FIXED
2 years ago
10 months ago

People

(Reporter: Thomas D. (currently busy elsewhere), Assigned: Thomas D. (currently busy elsewhere))

Tracking

(Blocks: 1 bug, {ux-consistency, ux-efficiency})

Trunk
ux-consistency, ux-efficiency
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Fixed by bug 1322032])

Attachments

(1 obsolete attachment)

+++ This bug was initially created as a clone of Bug #998312 +++

For all the reasons mentioned in Bug 998312 (with much detail), we surely also want to offer this keyboard shortcut to users:

Ctrl+I on selected and focused contact/list in contact side bar
--> Edit contact/list properties

Cmd/Ctrl+I is the MAC default shortcut for properties (like Alt+Enter on Windows/Linux).
We also offer both shortcuts in ABView of main AB (Directory pane needs to be done).
And those shortcuts can never do anything else on a selected and focused contact because that would violate the standards.

Richard, can I please have a thumbs-up...
(Assignee)

Comment 1

2 years ago
ui-r?paenglab --> comment 0
Flags: needinfo?(richard.marti)
This makes sense to be consistent.
Flags: needinfo?(richard.marti)
(Assignee)

Comment 3

2 years ago
Simple fix (tested working locally on Windows)...

https://dxr.mozilla.org/comm-central/source/mail/components/addrbook/content/abContactsPanel.js

> function contactsListOnKeyPress(aEvent)
> {
>   switch (aEvent.key) {
>     case "Enter":
>       if (aEvent.altKey) {
>         goDoCommand("cmd_properties");
>       }
      case "i":
        if (aEvent.ctrlKey) {
          goDoCommand("cmd_properties");
        }
    }
  }

Not sure how to fix this for MAC, but fixing for Windows also helps...
(Assignee)

Updated

2 years ago
Depends on: 1319409
(Assignee)

Updated

2 years ago
Assignee: nobody → bugzilla2007
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
Severity: normal → enhancement
(Assignee)

Updated

2 years ago
Depends on: 1320475
(Assignee)

Comment 4

2 years ago
Created attachment 8815896 [details] [diff] [review]
Patch: Ctrl+I for contacts sidebar item properties

Simple Fix!

Can even be landed at any time, those four lines don't hurt much.
Attachment #8815896 - Flags: ui-review?(richard.marti)
Attachment #8815896 - Flags: review?(richard.marti)
Comment on attachment 8815896 [details] [diff] [review]
Patch: Ctrl+I for contacts sidebar item properties

Looks good and simplicity of code should allow me to r+

ui-r isn't needed on this change.
Attachment #8815896 - Flags: ui-review?(richard.marti)
Attachment #8815896 - Flags: review?(richard.marti)
Attachment #8815896 - Flags: review+
(Assignee)

Comment 6

10 months ago
I fixed this in a better way in bug 1322032.

https://hg.mozilla.org/comm-central/diff/8f2f953f2ed3/mail/components/addrbook/content/abContactsPanel.xul#l1.50

+  <keyset id="keyset_abContactsPanel">
+    <!-- This key (key_delete) does not trigger any command, but it is used
+         only to show the hotkey on the corresponding menuitem. -->
+    <key id="key_delete" keycode="VK_DELETE"/>
+#ifdef XP_MACOSX
+    <key id="key_properties" modifiers="accel" key="&propertiesCmd.key;" command="cmd_properties"/>
+#else
+    <key id="key_properties" modifiers="alt" keycode="VK_RETURN" command="cmd_properties"/>
+    <key id="key_properties2" modifiers="accel" key="&propertiesCmd.key;" command="cmd_properties"/>    <!-- Ctrl+I -->
+#endif
+  </keyset>

So on Windows and Linux, for maximum convenience we allow both, Alt+Enter (Windows platform shortcut for properties) and Ctrl+I (Mac platform shortcut for properties).

I think someone mentioned that Alt+Enter is used for something else on MAC, or at least it might be, so MAC only has Ctrl+I.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
Whiteboard: [Fixed by bug 1322032]
(Assignee)

Updated

10 months ago
Attachment #8815896 - Attachment is obsolete: true
(Assignee)

Comment 7

10 months ago
So the last piece in the ux-consistency puzzle of cmd_properties keyboard shortcuts is bug 1319040, Alt+Enter for directory pane of AB.
See Also: → bug 1319040
(Assignee)

Updated

10 months ago
Depends on: 1322032
You need to log in before you can comment on or make changes to this bug.