D11150: Use Kirigami's spacing and icon sizes and remove own implementation

Malte Veerman noreply at phabricator.kde.org
Thu Mar 8 09:51:33 UTC 2018


malteveerman created this revision.
malteveerman added a reviewer: Amarok.
malteveerman added a project: Amarok.
malteveerman requested review of this revision.

REVISION SUMMARY
  This patch removes some redundant functionality from ContextView and makes the QML parts instead rely on KF5::Kirigami2. Also adds Kirigami as runtime package.

REPOSITORY
  R181 Amarok

BRANCH
  kf5

REVISION DETAIL
  https://phabricator.kde.org/D11150

AFFECTED FILES
  CMakeLists.txt
  src/context/ContextView.cpp
  src/context/ContextView.h
  src/context/applets/analyzer/package/contents/ui/main.qml
  src/context/applets/currenttrack/package/contents/ui/InfoItem.qml
  src/context/applets/currenttrack/package/contents/ui/StatsItem.qml
  src/context/applets/currenttrack/package/contents/ui/main.qml
  src/context/applets/info/package/contents/ui/main.qml
  src/context/applets/photos/package/contents/ui/main.qml
  src/context/applets/wikipedia/package/contents/ui/main.qml
  src/context/context_qml_package/contents/ui/main.qml
  src/context/context_qml_package/contents/ui/toolbar/AppletToolbar.qml
  src/context/context_qml_package/contents/ui/toolbar/AppletToolbarAddItem.qml
  src/context/context_qml_package/contents/ui/toolbar/AppletToolbarAppletItem.qml
  src/context/qml_plugin/Applet.qml

To: malteveerman, #amarok
Cc: #amarok, nhuisman, paul, yaohanchen, malteveerman, markey, cochise, tbettler, Smar, ricktimmis, asturmlechner, schweingruber, heikobecker
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20180308/28af0658/attachment.html>


More information about the Amarok-devel mailing list