D24361: [Style KCM] Port to new design
Kai Uwe Broulik
noreply at phabricator.kde.org
Wed Oct 2 13:02:52 BST 2019
broulik created this revision.
broulik added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
This ports the widget style KCM to use a grid view KCM with live preview thumbnails and inline configure button.
- Icon and tool bar settings have been moved into a popup, similar to Icons KCM, to leave more room for the rather large thumbnails.
- Overall cleanup of unused/defunct features and modernized codebase
- Support for non-Linux is dropped, i.e. it assumes "Breeze" as the default, no "native" (e.g. Mac or Windows style) as we want our apps to follow whatever other platform they run in, rather than offering this KCM there
TEST PLAN
F7496756: Screenshot_20191002_135630.png <https://phabricator.kde.org/F7496756>
"Advanced" settings
F7496757: Screenshot_20191002_135647.png <https://phabricator.kde.org/F7496757>
Widget preview glows on hover like they used to
F7496759: Screenshot_20191002_135707.png <https://phabricator.kde.org/F7496759>
- Applying style works live
- Applying icon and toolbar settings work live
- Configuring a style works and updates the preview live once you close the style config dialog
Two issues remaining
- Just found I broke the kcminit stuff: "Module "kcm_style" was not found or does not actually have a kcminit function"
- On destruction of the qtcurve style it seems to freeze/deadlock, when you close the kcm by clicking "OK" (can't reproduce the backtrace I had but the process doesn't quit)
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D24361
AFFECTED FILES
kcms/style/CMakeLists.txt
kcms/style/Messages.sh
kcms/style/kcmstyle.cpp
kcms/style/kcmstyle.h
kcms/style/package/contents/ui/EffectSettingsPopup.qml
kcms/style/package/contents/ui/main.qml
kcms/style/package/metadata.desktop
kcms/style/previewitem.cpp
kcms/style/previewitem.h
kcms/style/styleconfig.ui
kcms/style/stylesmodel.cpp
kcms/style/stylesmodel.h
To: broulik, #plasma, #vdg
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191002/8b790521/attachment.html>
More information about the Plasma-devel
mailing list