Closed
Bug 772005
Opened 13 years ago
Closed 13 years ago
Further improve notification of new chat messages
Categories
(Thunderbird :: Instant Messaging, defect)
Tracking
(thunderbird15 fixed, thunderbird16 fixed)
RESOLVED
FIXED
Thunderbird 17.0
People
(Reporter: andreasn, Assigned: Paenglab)
Details
Attachments
(6 files, 2 obsolete files)
52.56 KB,
image/gif
|
Details | |
29.90 KB,
image/png
|
Details | |
2.74 KB,
image/png
|
Details | |
24.90 KB,
image/png
|
Details | |
6.82 KB,
patch
|
mconley
:
review+
bwinton
:
ui-review+
bwinton
:
approval-comm-aurora+
bwinton
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
13.86 KB,
image/png
|
Details |
In bug #748358 we stopped opening a new tab and changed the icon when there were new unread chat messages.
JB suggested that we should try and do something more than just change the color (Win/Mac, Linux use a flare) to grab your attention. It would also be good to allow you to see the number of unread messages.
Reporter | ||
Comment 1•13 years ago
|
||
It's quite common to use a blinking icon in IM clients, so I would suggest to try that.
Reporter | ||
Comment 2•13 years ago
|
||
Mockup of animation
Reporter | ||
Comment 3•13 years ago
|
||
counter proposal (mac)
Comment 4•13 years ago
|
||
Blake - what are your thoughts on these? My feeling is that the pulsing could get a bit irritating, and I'd go for the badge if possible...
Comment 5•13 years ago
|
||
Yep, exactly that.
Updated•13 years ago
|
Assignee: nobody → mconley
Comment 6•13 years ago
|
||
First crack at this - I had to add a new XBL binding for a special toolbarbutton that has a "badge" element. I couldn't just toss a label inside a toolbarbutton, otherwise the icon / label would go away. :/
Positioning the badge is a little tricky, and I'm going to see if I can get Paenglab's help on this too.
Failing that, we can embed a badge into the unreadMessage icon, just without a count.
Comment 7•13 years ago
|
||
Paenglab:
Hey - would you have some time to help me out here too? I'm trying to position the red badge overtop of the Chat button icon, but it's rather tricky...hovering over the icon often causes the badge to move around, which is strange.
-Mike
Assignee | ||
Comment 8•13 years ago
|
||
What about this?
The badge is also shown in text only mode. Maybe it should be then different styled because the user don't want icons and the badge should look more like a text element.
What I saw during testing is when changing the button mode (text, icon only, text beside etc.) the badge value is deleted. This isn't important because it's unlikely someone plays a lot with the mode. But maybe it's a bug?
Attachment #644429 -
Attachment is obsolete: true
Assignee | ||
Comment 9•13 years ago
|
||
When are a lot of unread messages, everything after the badge is shifted to the right. I think it's okay like this because overlapping other elements shouldn't happen, especially in icon only mode it would then overlap the next button.
Comment 10•13 years ago
|
||
Richard:
This looks *awesome*, thank you so much. Just one issue - when in text-only mode for the toolbar, the badge hovers over top of the text, and causes a rather strange appearance. See my attachment.
With that fixed, I think it'll be ready for review.
-Mike
Assignee | ||
Comment 11•13 years ago
|
||
(In reply to Mike Conley (:mconley) from comment #10)
> This looks *awesome*, thank you so much. Just one issue - when in text-only
> mode for the toolbar, the badge hovers over top of the text, and causes a
> rather strange appearance. See my attachment.
You tried before the icon and text mode, didn't you? When you select then text only then the box-orient is vertical. I added a -moz-box-orient: horizontal; when in text mode to force the badge to be on the side.
> With that fixed, I think it'll be ready for review.
Okay, here it is.
Attachment #644628 -
Attachment is obsolete: true
Attachment #644962 -
Flags: ui-review?(bwinton)
Attachment #644962 -
Flags: review?(mconley)
Comment 12•13 years ago
|
||
Comment on attachment 644962 [details] [diff] [review]
Patch
Nice! ui-r=me!
Attachment #644962 -
Flags: ui-review?(bwinton) → ui-review+
Comment 13•13 years ago
|
||
Comment on attachment 644962 [details] [diff] [review]
Patch
Code looks good - let's roll with this.
Thanks so much Richard - this is fantastic work!
Attachment #644962 -
Flags: review?(mconley) → review+
Updated•13 years ago
|
OS: Linux → All
Updated•13 years ago
|
Attachment #644962 -
Flags: approval-comm-beta?
Attachment #644962 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 14•13 years ago
|
||
Assignee: mconley → richard.marti
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
Comment 15•13 years ago
|
||
Comment on attachment 644962 [details] [diff] [review]
Patch
This seems like something we want for the initial IM release.
Attachment #644962 -
Flags: approval-comm-beta?
Attachment #644962 -
Flags: approval-comm-beta+
Attachment #644962 -
Flags: approval-comm-aurora?
Attachment #644962 -
Flags: approval-comm-aurora+
Comment 16•13 years ago
|
||
comm-aurora: https://hg.mozilla.org/releases/comm-aurora/rev/6bbf30dd330f
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/105ee0a9ef2d
status-thunderbird15:
--- → fixed
status-thunderbird16:
--- → fixed
Comment 17•13 years ago
|
||
Comment 18•13 years ago
|
||
On my Macbook Air 13", the red badge looks a little bit pixelated and oval (i.s.o round). See attached picture.
Comment 19•13 years ago
|
||
Comment on attachment 647517 [details]
Pixelated & oval badge (MacBook Air 13")
For reference, here's the notification on Things, a native Mac app: http://dl.dropbox.com/u/2301433/Screenshots/redOval.png
Seems pretty oval to me, so I think our shape is okay. I'm not sure what to do to make it smoother. Perhaps we just live with it, since the number inside isn't the important thing, but rather the existence (or not) of the red button is.
Comment 20•13 years ago
|
||
I have a hard time understanding the behavior and meaning of the different signals. I am under the assumption that
- a blue envelope means 'there are new chat messages'
- a red badge means 'your name, or a tracked keyword (Twitter) is contained in one of the new chat messages', or 'you have a direct message'.
The behavior I see does not correspond to this description. Please see http://imgur.com/a/DMf8T for a screen shot of the toolbar notification and Chat panel report.
Comment 21•13 years ago
|
||
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #19)
> Comment on attachment 647517 [details]
> Pixelated & oval badge (MacBook Air 13")
>
> For reference, here's the notification on Things, a native Mac app:
> http://dl.dropbox.com/u/2301433/Screenshots/redOval.png
> Seems pretty oval to me, so I think our shape is okay.
Your oval is larger than tall. Mine is the opposite. My personal taste goes to your layout, but this maybe just my own taste...
Comment 22•12 years ago
|
||
As this bug seems general enough, I'd like to propose to send desktop notification when the status is 'Available' and Thunderbird is not focused. This is quite usual with IM application to notify new message on arrival (sender & message)
I think that with empathy/gnome3, the notification can be used to answer without going to the app - but that's overkill IMHO.
what do you think? should I open a seperated bug?
You need to log in
before you can comment on or make changes to this bug.
Description
•