Terminal

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

  1. fix-1393219
  2. Merge into 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
lp:~elementary-apps/pantheon-terminal/fix-1393219 updated
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
1=== modified file 'data/pantheon-terminal.desktop'
2--- data/pantheon-terminal.desktop 2014-08-09 17:05:43 +0000
3+++ data/pantheon-terminal.desktop 2014-11-30 18:06:30 +0000
4@@ -10,6 +10,7 @@
5 StartupNotify=true
6 X-GNOME-Gettext-Domain=pantheon-terminal
7 Keywords=command;prompt;cmd;commandline;run;
8+X-GNOME-UsesNotifications=true;
9 Actions=NewWindow;NewRootWindow;AboutDialog;
10
11 [Desktop Action NewWindow]

Subscribers

People subscribed via source and target branches