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 682268 - Notification on adding/removing favorites is not shown
Notification on adding/removing favorites is not shown
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: message-tray
3.5.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-08-20 13:57 UTC by António Fernandes
Modified: 2012-08-27 10:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Weird Artefact (22.71 KB, image/png)
2012-08-20 13:57 UTC, António Fernandes
  Details
overview: Fix ShellInfo.setMessage() (1.31 KB, patch)
2012-08-20 14:17 UTC, Florian Müllner
committed Details | Review

Description António Fernandes 2012-08-20 13:57:43 UTC
Created attachment 221847 [details]
Weird Artefact

Steps to reproduce:

Enter Activities overview. Add or remove an application from favourites (either from its context menu or by drag and drop).

Actual results:

The light bulb source icon is added to the tray, but no notification comes up.
The source icon stays on the tray indefinitely until I do one of the following actions:
1) right click on the source then click on "Remove"
2) right click on the source then click on "Open"
3) right click on the source then left click on the source; this action produces the attached artefact.

Expected result:

"[Application] has been removed from favourites." Notification is shown. After some seconds, notification and source icon go away.

gnome-shell built via jhbuild on Fedora 17
Comment 1 Florian Müllner 2012-08-20 14:17:05 UTC
Created attachment 221851 [details] [review]
overview: Fix ShellInfo.setMessage()

Commit 22eea750 made info messages show up in the lock screen, but
as setShowWhenLocked() throws an exception when called on non-transient
notifications, the transient hint has to be set first.
Comment 2 António Fernandes 2012-08-20 14:41:16 UTC
(In reply to comment #1)

Yes, this fixed the issue for me.

The whole animation of message tray sliding down and notification banner siding up (and then the reverse) is a little annoying, but I guess that's a separate issue.

Also, the notification is already "expanded", i.e., the "Undo" button is already visible without before moving the pointer over the banner (however, the close button is not). Is this intended?
Comment 3 drago01 2012-08-27 09:45:14 UTC
Review of attachment 221851 [details] [review]:

Looks good.
Comment 4 Florian Müllner 2012-08-27 10:16:21 UTC
Attachment 221851 [details] pushed as 417941a - overview: Fix ShellInfo.setMessage()