Improve keyboard interactions with the status selector of the chat toolbar

RESOLVED FIXED in Thunderbird 17.0

Status

Thunderbird
Instant Messaging
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: florian)

Tracking

Trunk
Thunderbird 17.0

Thunderbird Tracking Flags

(thunderbird15 fixed, thunderbird16 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Bug 735658 fixed the appearance of the chat status selector, but there are a few more details that should be fixed:
- When there's no configured account, the selector should be (and look) disabled (mentioned in bug 735701 comment 1).
- It should be visible if the selector is focused (for people using the tab key to navigate between focusable elements) (mentioned in point 3 or bug 735292 comment 5; and maybe some of the following comments).
- Some keyboard interactions may be better. When the selector is focused, currently pressing enter starts the edit of the status message. Shouldn't the down arrow open the status type popup menu?
- After finishing to edit the status message, there's no focused element in the document any more (pressing the tab key does nothing); I think we need a focus call, like the one we have when finishing the rename of a contact.
- When the status type is "Offline", the status message isn't editable, so we shouldn't display the textbox border while hovering it.
(Assignee)

Comment 1

5 years ago
Created attachment 650953 [details] [diff] [review]
Patch

This does all the things listed in comment 0 + disables the "Show Accounts" button of the chat toolbar if there's no account. The "Get started" button in the middle of the main part of the chat tab in that case is much more visible, and opens directly the account wizard instead of the accounts status dialog which could be confusing when there's no account (it's not broken though).
Assignee: nobody → florian
Attachment #650953 - Flags: ui-review?(bwinton)
Attachment #650953 - Flags: review?(bwinton)
Comment on attachment 650953 [details] [diff] [review]
Patch

Yeah, this seems to make sense.  I'ld like the changes to be documented somewhere more visible than a random bug, though.  ;)
ui-r=me.

>+++ b/mail/components/im/content/chat.css
>@@ -57,25 +57,25 @@ browser[type="content-conversation"] {
>-#statusTypeIcon {
>+#statusTypeIcon:not([disabled]) {
>   cursor: pointer;
> }

I wonder if this is the right thing to do, since we don't do it for any other buttons…

Other than that, it seems fine to me.

r=me.

Thanks,
Blake,
Attachment #650953 - Flags: ui-review?(bwinton)
Attachment #650953 - Flags: ui-review+
Attachment #650953 - Flags: review?(bwinton)
Attachment #650953 - Flags: review+
Comment on attachment 650953 [details] [diff] [review]
Patch

Yadda yadda IM approval request.
Attachment #650953 - Flags: approval-comm-beta?
Attachment #650953 - Flags: approval-comm-aurora?
Attachment #650953 - Flags: approval-comm-beta?
Attachment #650953 - Flags: approval-comm-beta+
Attachment #650953 - Flags: approval-comm-aurora?
Attachment #650953 - Flags: approval-comm-aurora+
comm-central: https://hg.mozilla.org/comm-central/rev/00b6fe99218f
comm-aurora: https://hg.mozilla.org/releases/comm-aurora/rev/6619227bcb33
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/4577a7128713
Status: NEW → RESOLVED
Last Resolved: 5 years ago
status-thunderbird15: --- → fixed
status-thunderbird16: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.