[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [plasma-desktop/Plasma/5.6] applets/kimpanel/package/contents/ui: kimpanel: use Plasma.IconItem inst
From:       Weng Xuetian <wengxt () gmail ! com>
Date:       2016-03-05 4:07:49
Message-ID: E1ac3VN-0002l3-I8 () scm ! kde ! org
[Download RAW message or body]

Git commit b270e277599cc6812777e44c2ac9113bcce4c30f by Weng Xuetian.
Committed on 05/03/2016 at 04:05.
Pushed by xuetianweng into branch 'Plasma/5.6'.

kimpanel: use Plasma.IconItem instead of QML Image

The initial reason that we don't use Plasma.IconItem here is because
animation of Plasma.IconItem cannot be disabled. This reason is not
valid anymore, and this will enable us to make use to breeze stylesheet
based icon. Also absolute path icon is also tested with fcitx-mozc.

M  +3    -4    applets/kimpanel/package/contents/ui/StatusIcon.qml

http://commits.kde.org/plasma-desktop/b270e277599cc6812777e44c2ac9113bcce4c=
30f

diff --git a/applets/kimpanel/package/contents/ui/StatusIcon.qml b/applets/=
kimpanel/package/contents/ui/StatusIcon.qml
index 01345ed..bf3afc7 100644
--- a/applets/kimpanel/package/contents/ui/StatusIcon.qml
+++ b/applets/kimpanel/package/contents/ui/StatusIcon.qml
@@ -53,16 +53,15 @@ Item {
         return p;
     }
 =

-    Image {
+    PlasmaCore.IconItem {
         id: imageIcon
         anchors.centerIn: parent
         width: iconSize
         height: iconSize
         scale: mouseArea.pressed ? 0.9 : 1
-        source: iconPath(statusIcon.icon)
+        source: statusIcon.icon
         visible: statusIcon.icon.length > 0
-        sourceSize.width: iconSize
-        sourceSize.height: iconSize
+        animated: false
         // active: mouseArea.containsMouse
     }
     PlasmaComponents.Label {
[prev in list] [next in list] [prev in thread] [next in thread]