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 687785 - Indicate updated items in the tray
Indicate updated items in the tray
Status: RESOLVED OBSOLETE
Product: gnome-shell
Classification: Core
Component: message-tray
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-11-06 17:05 UTC by Allan Day
Modified: 2015-02-26 22:54 UTC
See Also:
GNOME target: ---
GNOME version: 3.7/3.8


Attachments
Look with initial focus (157.88 KB, image/jpeg)
2012-11-08 10:10 UTC, Stéphane Démurget
  Details
Look without focus (159.77 KB, image/jpeg)
2012-11-08 10:11 UTC, Stéphane Démurget
  Details
messageTray: indicate updated items (2.00 KB, patch)
2012-11-21 15:09 UTC, Stéphane Démurget
none Details | Review

Description Allan Day 2012-11-06 17:05:15 UTC
It would be useful to indicate which tray items have been updated. This will provide a reminder of which items still need attention. It will also provide an indication of when new messages come in while the tray is open.

There is an old patch that used to do this [1]. I've been told that it shouldn't be too hard to update it. :)

This bug ties in with other plans that we have for the message tray [2].

[1] https://bugzilla.gnome.org/attachment.cgi?id=182203&action=edit
[2] https://live.gnome.org/GnomeShell/Design/Guidelines/MessageTray/#A3.8_Roadmap
Comment 1 Stéphane Démurget 2012-11-07 15:44:27 UTC
I'll try this one.
Comment 2 Stéphane Démurget 2012-11-08 10:10:54 UTC
Created attachment 228449 [details]
Look with initial focus
Comment 3 Stéphane Démurget 2012-11-08 10:11:19 UTC
Created attachment 228450 [details]
Look without focus
Comment 4 Stéphane Démurget 2012-11-08 10:18:03 UTC
Questions for designers:
- the new wireframe glow seems to be very thin, is there's a mockup of how the glow would render?
- otherwise, which pixmap do you recommend to use (I used the running-indicator one)
- with the initial focus border, we do not see the glow

For the moment, the glow takes the space of the "focus border" which is tiny. I could put the glow right at the bottom of the message tray but would that be enough? Isn't the pixmap too thick too?
Comment 5 Allan Day 2012-11-08 11:23:50 UTC
Thanks for the screenshots Stéphane. I agree that it is hard to make out the glow effect when an item is focused. Maybe Jakub can advise here.
Comment 6 Stéphane Démurget 2012-11-16 06:30:32 UTC
Jakub: any hint?
Comment 7 Jakub Steiner 2012-11-16 11:22:07 UTC
The chosen visuals are fine with me.

But ideally I would lower the opacity of resident messages that have been seen, rather than increase visibility of unseen/new ones. Feels like we are shouting with light everywhere.
Comment 8 Jasper St. Pierre (not reading bugmail) 2012-11-16 17:31:37 UTC
(In reply to comment #7)
> The chosen visuals are fine with me.

They're barely visible!
Comment 9 Lapo Calamandrei 2012-11-16 18:16:52 UTC
I'm not sure about that one, we use that visual indication for selected or active items in other places, it's confusing used like that to me.
Comment 10 Stéphane Démurget 2012-11-17 15:32:03 UTC
We need a better visual indication, that's for sure. Maybe we could do something with the badge instead of the whole icon as an alternative? gray out non-updated items, or color + spiky border change like in http://git.gnome.org/browse/gnome-packagekit/plain/data/icons/48x48/status/pk-update-normal.png the updated ones?
Comment 11 Lapo Calamandrei 2012-11-17 16:34:35 UTC
Stéphane I think everything along those lines would be too bold, maybe an icon overlay like a "new" star in the top right corner would work, this would need repositioning the number of messages indicator in the bottom right corner though.
Comment 12 Allan Day 2012-11-20 13:27:30 UTC
Stéphane, could you post your work in progress? I'd like to try it and have a play around.
Comment 13 Stéphane Démurget 2012-11-21 15:09:23 UTC
Created attachment 229581 [details] [review]
messageTray: indicate updated items

This provides a reminder of which items still need attention. It also
provides an indication of when new messages come in while the tray is open.
Comment 14 Stéphane Démurget 2012-11-21 15:19:07 UTC
Here is my preliminary patch, not meant for code review, but I still have a technical question: is it allowed for a source to send spurious (dummy) count updates? Is is the role of the indicator or the source or nobody to guard against this? just asking in case I should keep a counter with the last value... 

Allan: I've adjusted the background-size for the running indicator to be under the focus rectangle, but I find image too tall.
Comment 15 Giovanni Campagna 2012-11-28 19:59:11 UTC
(In reply to comment #14)
> Here is my preliminary patch, not meant for code review, but I still have a
> technical question: is it allowed for a source to send spurious (dummy) count
> updates? Is is the role of the indicator or the source or nobody to guard
> against this? just asking in case I should keep a counter with the last
> value... 

count-updated should behave like notify::count would if count was a GObject property, ie. the source should try as hard as possible to avoid emitting it when it's not necessary, but the summary item should cope with it not changing.

There is one exception here: count-updated is notify::count || notify::countVisible, so you'll get some spurious updates as notifications become acknowledged.

Which makes me think: should the indicator all notifications, or just notifications that were never seen (acknowledged, in code) by the user?
Because in the former case, to me we're duplicating the message indicator, while in the latter case, this indicator will be rarely seen, as notifications are usually presented first in banner form.
Maybe we need a third flag and counter, reporting if the notification was acted upon in any way (expanded, action button clicked, focused...) since last updated?
Comment 16 Allan Day 2012-12-14 09:54:48 UTC
Thanks for the patch Stéphane. Trying this out for myself, the prelight effect doesn't seem right. We generally use it to communicate that something is active, but that isn't the case here.

We need another idea for the visuals here. I still like the concept behind this bug though.
Comment 17 Allan Day 2015-02-26 22:54:34 UTC
With the new notifications design [1], this bug is no longer needed.

[1] https://wiki.gnome.org/Design/OS/Notifications/