Merge lp:~laney/telephony-service/humanity-not-gnome into lp:telephony-service

Proposed by Iain Lane on 2015-01-30
Status: Superseded
Proposed branch: lp:~laney/telephony-service/humanity-not-gnome
Merge into: lp:telephony-service
Diff against target: 31 lines (+5/-4)
2 files modified
debian/telephony-service.install (+3/-3)
indicator/icons/CMakeLists.txt (+2/-1)
To merge this branch: bzr merge lp:~laney/telephony-service/humanity-not-gnome
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) 2015-01-30 Approve on 2015-06-22
Dmitry Shachnev 2015-01-30 Approve on 2015-01-30
Review via email: mp+248128@code.launchpad.net

This proposal has been superseded by a proposal from 2015-07-06.

Commit Message

We don't inherit from the 'gnome' icon theme any more, so our icons weren't being found in a default Unity session using ubuntu-mono-dark. We do, however, inherit from Humanity, so put the icons under this theme.

Description of the Change

To post a comment you must log in.
Dmitry Shachnev (mitya57) wrote :

Not tested, but LGTM

review: Approve
Iain Lane (laney) wrote :

Can someone from the telephony team look at this?

Gustavo Pichorim Boiko (boiko) wrote :

Looks good!

review: Approve

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/telephony-service.install'
2--- debian/telephony-service.install 2014-03-24 18:53:27 +0000
3+++ debian/telephony-service.install 2015-01-30 15:30:53 +0000
4@@ -8,9 +8,9 @@
5 usr/share/icons/ubuntu-mono-dark/status/*/indicator-call.svg
6 usr/share/icons/ubuntu-mono-light/status/*/indicator-call.svg
7 usr/share/locale/*/LC_MESSAGES/telephony-service.mo
8-usr/share/notify-osd/icons/gnome/scalable/status/notification-group-call.svg
9-usr/share/notify-osd/icons/gnome/scalable/status/notification-unavailable-image-call.svg
10-usr/share/notify-osd/icons/gnome/scalable/status/notification-unknown-call.svg
11+usr/share/notify-osd/icons/Humanity/scalable/status/notification-group-call.svg
12+usr/share/notify-osd/icons/Humanity/scalable/status/notification-unavailable-image-call.svg
13+usr/share/notify-osd/icons/Humanity/scalable/status/notification-unknown-call.svg
14 usr/share/polkit-1/actions/*TelephonyService*.policy
15 usr/share/telepathy/clients/TelephonyService*.client
16 usr/share/telephony-service/assets/*
17
18=== modified file 'indicator/icons/CMakeLists.txt'
19--- indicator/icons/CMakeLists.txt 2012-07-20 14:14:47 +0000
20+++ indicator/icons/CMakeLists.txt 2015-01-30 15:30:53 +0000
21@@ -1,3 +1,4 @@
22+set (icon-theme Humanity)
23
24 set(icons
25 notification-group-call.svg
26@@ -5,4 +6,4 @@
27 notification-unknown-call.svg
28 )
29
30-install(FILES ${icons} DESTINATION share/notify-osd/icons/gnome/scalable/status)
31+install(FILES ${icons} DESTINATION share/notify-osd/icons/${icon-theme}/scalable/status)

Subscribers

People subscribed via source and target branches

to all changes: