Merge lp:~elementary-apps/pantheon-terminal/fix-1393219 into lp:~elementary-apps/pantheon-terminal/trunk

Proposed by Danielle Foré
Status: Merged
Approved by: David Gomes
Approved revision: 654
Merged at revision: 654
Proposed branch: lp:~elementary-apps/pantheon-terminal/fix-1393219
Merge into: lp:~elementary-apps/pantheon-terminal/trunk
Diff against target: 11 lines (+1/-0)
1 file modified
data/pantheon-terminal.desktop (+1/-0)
To merge this branch: bzr merge lp:~elementary-apps/pantheon-terminal/fix-1393219
Reviewer Review Type Date Requested Status
David Gomes (community) Approve
Review via email: mp+241897@code.launchpad.net

Commit message

adds "X-GNOME-UsesNotifications=true" to the .desktop

Description of the change

adds "X-GNOME-UsesNotifications=true" to the .desktop so that notification settings can discover Terminal

To post a comment you must log in.
Revision history for this message
David Gomes (davidgomes) wrote :

What about a semi-colon?

review: Needs Fixing
654. By artem-anufrij

Add semi-colon

Revision history for this message
Artem Anufrij (artem-anufrij) wrote :

@David:

semi-colon was added.

Revision history for this message
David Gomes (davidgomes) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/pantheon-terminal.desktop'
--- data/pantheon-terminal.desktop 2014-08-09 17:05:43 +0000
+++ data/pantheon-terminal.desktop 2014-11-30 18:06:30 +0000
@@ -10,6 +10,7 @@
10StartupNotify=true10StartupNotify=true
11X-GNOME-Gettext-Domain=pantheon-terminal11X-GNOME-Gettext-Domain=pantheon-terminal
12Keywords=command;prompt;cmd;commandline;run;12Keywords=command;prompt;cmd;commandline;run;
13X-GNOME-UsesNotifications=true;
13Actions=NewWindow;NewRootWindow;AboutDialog;14Actions=NewWindow;NewRootWindow;AboutDialog;
1415
15[Desktop Action NewWindow]16[Desktop Action NewWindow]

Subscribers

People subscribed via source and target branches