D14693: Turn qqc2-desktop-style in a proper Qt plugin

Emmanuel Lepage Vallee noreply at phabricator.kde.org
Wed Aug 8 14:52:22 BST 2018


lepagevalleeemmanuel created this revision.
lepagevalleeemmanuel added a reviewer: mart.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
lepagevalleeemmanuel requested review of this revision.

REVISION SUMMARY
  Add support for static Qt plugins.
  
  This style can now replace the default style in applications built
  with the static Qt SDK. It allows better encapsulation on the desktop
  and makes it easier to try applications on Plasma Mobile.
  
  - Remove dead code **
  
  There was an unused class
  
  - Gnerate a QML QRC at build-time **
  
  Since commit d0e3daafdbd <https://phabricator.kde.org/R858:d0e3daafdbdd858a5ee6e6b0b2de6a7a39ff27f1>, all QML files are now preprocessed. This
  renders the QRC file introduced in a few commits ago invalid since
  it uses the non-preprocessed files.
  
  The commit also reorder the CMake targets to make sure the files
  are preprecessed before the plugin is built.
  
  - Rename org.kde.qqc2desktopstyle.private to org.kde.desktop.private **
  
  Submodules are supposed to have a consistent URI
  
  - Add an option to enable or disable plasma integration. **
  
  The way the plugin is built is incompatible with static AppImage and
  probably other self contained file format.
  
  - Use the same coding style for all "private" import **
  
  First, there was a typo in my previous commit. But the main problem
  was that even like that, mixing org.kde.desktop/private,
  org.kde.desktop.private and "private" is not very portable when
  switching between static and dynamic mode.

TEST PLAN
  Not much. Works for me. It will need other users to validate it didn't regress anything for them. I would suggest not merging this as-is

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

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

AFFECTED FILES
  CMakeLists.txt
  org.kde.desktop/Button.qml
  org.kde.desktop/CheckBox.qml
  org.kde.desktop/CheckDelegate.qml
  org.kde.desktop/CheckIndicator.qml
  org.kde.desktop/ComboBox.qml
  org.kde.desktop/Dial.qml
  org.kde.desktop/ItemDelegate.qml
  org.kde.desktop/Label.qml
  org.kde.desktop/ProgressBar.qml
  org.kde.desktop/RadioButton.qml
  org.kde.desktop/RadioDelegate.qml
  org.kde.desktop/ScrollBar.qml
  org.kde.desktop/ScrollView.qml
  org.kde.desktop/Slider.qml
  org.kde.desktop/SpinBox.qml
  org.kde.desktop/SwitchDelegate.qml
  org.kde.desktop/SwitchIndicator.qml
  org.kde.desktop/TabBar.qml
  org.kde.desktop/TabButton.qml
  org.kde.desktop/TextArea.qml
  org.kde.desktop/TextField.qml
  org.kde.desktop/ToolButton.qml
  org.kde.desktop/private/FocusRect.qml
  org.kde.desktop/private/qmldir
  org.kde.desktop/qmldir
  plugin/CMakeLists.txt
  plugin/kquickstyleitem.cpp
  plugin/kquickstyleitem_p.h
  plugin/qqc2desktopstyleplugin.cpp
  style.qrc.in

To: lepagevalleeemmanuel, mart
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180808/2bb026d3/attachment-0001.html>


More information about the Plasma-devel mailing list