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 677907 - Indicate that Evolution is being launched when "Open Calendar" is clicked
Indicate that Evolution is being launched when "Open Calendar" is clicked
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: calendar
unspecified
Other Linux
: Normal normal
: ---
Assigned To: David Zeuthen (not reading bugmail)
gnome-shell-maint
: 644077 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-06-12 01:36 UTC by Marina Zhurakhinskaya
Modified: 2012-10-24 06:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
calendar-server: Add .desktop file for evolution calendar (3.58 KB, patch)
2012-07-03 22:13 UTC, Florian Müllner
committed Details | Review
dateMenu: Use .desktop file to launch calendar (1.54 KB, patch)
2012-07-03 22:13 UTC, Florian Müllner
committed Details | Review

Description Marina Zhurakhinskaya 2012-06-12 01:36:02 UTC
It takes about 20 seconds to launch Evolution the first time "Open Calendar" is clicked and about 5 seconds each subsequent time. Switching to an already open Evolution Calendar window when "Open Calendar" is clicked also takes about 5 seconds. During this time, the user has no indication that the system is doing anything. Provide this indication by showing the spinner along with the Evolution application name in the panel, the same way it's done when Evolution is being launched from the Overview. Also, switch to the Evolution window immediately if Evolution is already running.
Comment 1 André Klapper 2012-06-13 15:47:36 UTC
[Setting the "Version" field is welcome... (in general)]
Comment 2 Matthias Clasen 2012-06-15 14:29:51 UTC
Why don't we get startup notification in this case ?
Comment 3 Florian Müllner 2012-06-15 15:23:23 UTC
(In reply to comment #2)
> Why don't we get startup notification in this case ?

Because we don't launch it via a .desktop file :(

Maybe we should add a (NoDisplay) evolution-calendar.desktop file for this?
Comment 4 Matthias Clasen 2012-06-15 15:49:30 UTC
If that solves this issue, sounds good to me.
Comment 5 Matthias Clasen 2012-07-03 20:56:31 UTC
The evolution guys prefer the desktop file to live with gnome-shell-calendar-server.
Comment 6 Matthias Clasen 2012-07-03 20:57:07 UTC
Marina, can you fix this ?
Comment 7 Matthias Clasen 2012-07-03 20:59:03 UTC
Matt says that just making the Exec line

 evolution --component calendar

should be good enough

The desktop file should be NoDisplay, I guess.
Comment 8 Florian Müllner 2012-07-03 22:13:02 UTC
Created attachment 217972 [details] [review]
calendar-server: Add .desktop file for evolution calendar

When selecting "Open Calendar" in the date menu, the configured
application is launched via command line, so we don't get any
startup notification. In order to fix the issue at least for our
default calendar, add a hidden .desktop file for evolution's
calendar component.
Comment 9 Florian Müllner 2012-07-03 22:13:08 UTC
Created attachment 217973 [details] [review]
dateMenu: Use .desktop file to launch calendar

When selecting "Open Calendar" in the date menu, the configured
application is launched via command line, so we don't get any
startup notification. If Evolution is used as calendar application,
launch it via the .desktop file added by the last commit instead in
order to fix the issue.
Comment 10 Florian Müllner 2012-07-03 22:14:19 UTC
(sorry, I had this actually lying around in my local tree; updated to install the .desktop file ourself rather than patching evolution)
Comment 11 Jasper St. Pierre (not reading bugmail) 2012-07-03 22:19:59 UTC
Review of attachment 217973 [details] [review]:

::: js/ui/dateMenu.js
@@ +171,3 @@
             // TODO: pass the selected day
+            let app = Shell.AppSystem.get_default().lookup_app('evolution-calendar.desktop');
+            app.launch(global.get_current_time(), [], -1, null);

Shouldn't this be activate?
Comment 12 Jasper St. Pierre (not reading bugmail) 2012-07-03 22:21:03 UTC
Review of attachment 217972 [details] [review]:

Sure.
Comment 13 Florian Müllner 2012-07-03 22:25:57 UTC
Attachment 217972 [details] pushed as 20769f6 - calendar-server: Add .desktop file for evolution calendar
Attachment 217973 [details] pushed as 66adeef - dateMenu: Use .desktop file to launch calendar

Pushed with s/launch/activate/
Comment 14 Florian Müllner 2012-10-24 06:51:32 UTC
*** Bug 644077 has been marked as a duplicate of this bug. ***