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 617666 - Doesn't switch off startup-notification
Doesn't switch off startup-notification
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
3.13.x
Other All
: Normal normal
: ---
Assigned To: Orca Maintainers
Orca Maintainers
3.0!
Depends on:
Blocks:
 
 
Reported: 2010-05-04 16:32 UTC by Bastien Nocera
Modified: 2017-08-16 14:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove unneeded try/except around Gdk.notify_startup_complete() (1.25 KB, patch)
2017-08-16 13:19 UTC, Bastien Nocera
committed Details | Review

Description Bastien Nocera 2010-05-04 16:32:05 UTC
1. In GDM, turn on Orca in the a11y preferences
2. Switch orca off
3. Switch it on again, and notice that the waiting cursor carries on spinning for a while

Orca should be finishing the startup notification with gdk_notify_startup_complete().
Comment 1 Joanmarie Diggs (IRC: joanie) 2010-07-05 02:45:49 UTC
(3.0 Planning Spam-o-rama. Sorry!)
Comment 2 Bastien Nocera 2014-08-31 19:26:37 UTC
Still a problem in 3.13. See also bug 735761.
Comment 3 Joanmarie Diggs (IRC: joanie) 2015-12-02 17:50:55 UTC
I've never been able to reproduce this. Since bug 735761 didn't remove the .desktop file in order to accommodate/not break expectations of/whatever non-GNOME environments, Orca (commit 198bd5bdea) now calls gdk_notify_startup_complete(). I don't see any difference, but since I wasn't ever able to reproduce this issue.... <insert shrug here>

Bastien, is there anything different/else I should be doing to resolve this issue?
Comment 4 Bastien Nocera 2017-08-16 13:17:13 UTC
I don't remember how it was started on older versions of GNOME, but I'm guessing that it might be through gnome-session, and g_app_info..() which would automatically use startup notification if available.

A couple of problems though:
- notifications have never worked properly on Wayland (see bug 768531)
- gdm now uses Wayland by default
- orca doesn't ship a .desktop file that gnome-shell could associate with the application being launched

The changes in that commit should be enough, though I simplified them somewhat in the attached patch. Fine to close afterwards.
Comment 5 Bastien Nocera 2017-08-16 13:19:08 UTC
Created attachment 357725 [details] [review]
Remove unneeded try/except around Gdk.notify_startup_complete()

Fix up 198bd5bdea3765eedab4299909cc4e2d7dd5e78c as the call is
always successful.
Comment 6 Joanmarie Diggs (IRC: joanie) 2017-08-16 14:37:35 UTC
Comment on attachment 357725 [details] [review]
Remove unneeded try/except around Gdk.notify_startup_complete()

Ok, thanks!
Comment 7 Bastien Nocera 2017-08-16 14:40:18 UTC
Attachment 357725 [details] pushed as c8d223e - Remove unneeded try/except around Gdk.notify_startup_complete()