Merge lp:~seb128/ubuntu-system-settings/system-update-logo into lp:ubuntu-system-settings

Proposed by Sebastien Bacher
Status: Merged
Approved by: Ken VanDine
Approved revision: 1507
Merged at revision: 1522
Proposed branch: lp:~seb128/ubuntu-system-settings/system-update-logo
Merge into: lp:ubuntu-system-settings
Diff against target: 43 lines (+5/-3)
3 files modified
plugins/system-update/CMakeLists.txt (+1/-1)
plugins/system-update/PageComponent.qml (+3/-1)
plugins/system-update/system_update.cpp (+1/-1)
To merge this branch: bzr merge lp:~seb128/ubuntu-system-settings/system-update-logo
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+269243@code.launchpad.net

Commit message

[system-update] use scalable distributor-logo icon from the theme

Description of the change

[system-update] use scalable distributor-logo icon from the theme

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks great, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/system-update/CMakeLists.txt'
2--- plugins/system-update/CMakeLists.txt 2014-10-22 19:37:42 +0000
3+++ plugins/system-update/CMakeLists.txt 2015-08-26 16:46:02 +0000
4@@ -25,7 +25,7 @@
5 install(FILES ${QML_SOURCES} DESTINATION ${PLUGIN_QML_DIR}/system-update)
6
7 install(FILES system-update.settings DESTINATION ${PLUGIN_MANIFEST_DIR})
8-install(FILES images/settings-system-update.svg images/distributor-logo.png DESTINATION ${PLUGIN_MANIFEST_DIR}/icons)
9+install(FILES images/settings-system-update.svg DESTINATION ${PLUGIN_MANIFEST_DIR}/icons)
10
11
12 set(QML_SOURCES_NOTIFICATION EntryComponent.qml)
13
14=== modified file 'plugins/system-update/PageComponent.qml'
15--- plugins/system-update/PageComponent.qml 2015-06-29 13:06:41 +0000
16+++ plugins/system-update/PageComponent.qml 2015-08-26 16:46:02 +0000
17@@ -714,8 +714,10 @@
18 spacing: units.gu(2)
19
20 Image {
21- source: Qt.resolvedUrl("file:///usr/share/ubuntu/settings/system/icons/distributor-logo.png")
22+ source: Qt.resolvedUrl("file:///usr/share/icons/suru/places/scalable/distributor-logo.svg")
23 anchors.horizontalCenter: parent.horizontalCenter
24+ height: width
25+ width: 96
26 NumberAnimation on rotation {
27 from: 0
28 to: 360
29
30=== removed file 'plugins/system-update/images/distributor-logo.png'
31Binary files plugins/system-update/images/distributor-logo.png 2014-01-23 19:29:34 +0000 and plugins/system-update/images/distributor-logo.png 1970-01-01 00:00:00 +0000 differ
32=== modified file 'plugins/system-update/system_update.cpp'
33--- plugins/system-update/system_update.cpp 2015-07-13 18:26:10 +0000
34+++ plugins/system-update/system_update.cpp 2015-08-26 16:46:02 +0000
35@@ -232,7 +232,7 @@
36 update->setSelected(downloading);
37 update->setUpdateAvailable(isAvailable);
38 update->setLastUpdateDate(lastUpdateDate);
39- update->setIconUrl(QString("file:///usr/share/ubuntu/settings/system/icons/distributor-logo.png"));
40+ update->setIconUrl(QString("file:///usr/share/icons/suru/places/scalable/distributor-logo.svg"));
41
42 if (update->updateRequired()) {
43 Q_EMIT updateAvailable(packageName, update);

Subscribers

People subscribed via source and target branches