<table><tr><td style="">broulik updated this revision to Diff 50123.<br />broulik retitled this revision from "WIP: [Colors KCM] Port to new design" to "[Colors KCM] Port to new design".<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-wzxcic3idijvo7g/">(Show Details)</a><br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4dbvifkiznzdfqx/">(Show Details)</a>
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12278">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">This removes the "Apply to non-Qt applications checkbox" which isn't something one should uncheck</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Overall the user experience has been streamlined and simplified a lot:<br />
* The "Default" theme option has been dropped in favor of having the "Defaults" button revert the selected theme to Breeze. While technically the old code made it read the hardcoded default colors in `KColorScheme` (which cause the window decoration to turn blue as it cannot write into KWin config like the theme files can), this change makes most sense from a UX POV.<br />
* The "Current" theme option has also been removed. Technically, when applying a theme the colors are copied into kdeglobals, so you could have a custom theme that is not an actual `.colors` file on disk. However, this is imho quite a niche usecase. Ideally, we showed a "Custom" theme as soon as the actual theme diverges from any theme file installed but that would require tediously comparing dozens of settings values which I don't think is feasible. At least when the color scheme name set in `kdeglobals` does not exist, a warning is now displayed.<br />
* The "Apply to non-Qt applications checkbox" which isn't something one would want to uncheck has been removed. It is still read from `kcmdisplayrc` for those who really want to disable it but there is no user-visible checkbox anymore.<br />
<br />
KColorSchemeEditor is now completely disentangled from the KCM and is merely launched as separate process:<br />
* When editing a system scheme, upon clicking "Save" the user is prompted to type a new scheme name. This ensures that any custom scheme is always present on disk reducing the need for a "Current" entry. When the dialog is then closed, the newly saved theme is selected</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">It is still read from config file and honored but there's no in-your-face checkbox for that anymore</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* When editing a user scheme, upon clicking "Save" the scheme is updated with any changes made in the dialog (different behavior from when kcolorschemeeditor is launched standalone, where "Save" is always essentially "Save As")<br />
<br />
The rewrite also comes with all the goodies we got in the other new KCMs, such as the ability to drop a `.colors` file into the view to install it or undo deletion until you apply your changes</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Closes T7243<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Revision 1<br />
{F5810572}<br />
Revision 2<br />
{F5819205}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6563933}</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">TODO:<br />
* Make restore defaults work</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Minor issues:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Figure out the "Current" theme stuff</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* KColorSchemeEditor isn't modal to the KCM</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Make edit work (the edit dialog is quite entangled with</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* KColorSchemeEditor lacks an Apply button so when launched from</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">code, I thought it was completely standalone nowadays)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">KCM you always have to do Save and Close instead.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D12278?vs=32881&id=50123">https://phabricator.kde.org/D12278?vs=32881&id=50123</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12278">https://phabricator.kde.org/D12278</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/colors/CMakeLists.txt<br />
kcms/colors/Messages.sh<br />
kcms/colors/TODO<br />
kcms/colors/colors.cpp<br />
kcms/colors/colors.desktop<br />
kcms/colors/colors.h<br />
kcms/colors/colorscm.cpp<br />
kcms/colors/colorscm.h<br />
kcms/colors/colorsettings.ui<br />
kcms/colors/editor/CMakeLists.txt<br />
kcms/colors/editor/colorsettings.ui<br />
kcms/colors/editor/kcolorschemeeditor.cpp<br />
kcms/colors/editor/org.kde.kcolorschemeeditor.desktop<br />
kcms/colors/editor/preview.ui<br />
kcms/colors/editor/previewwidget.cpp<br />
kcms/colors/editor/previewwidget.h<br />
kcms/colors/editor/scmeditorcolors.cpp<br />
kcms/colors/editor/scmeditorcolors.h<br />
kcms/colors/editor/scmeditorcolors.ui<br />
kcms/colors/editor/scmeditordialog.cpp<br />
kcms/colors/editor/scmeditordialog.h<br />
kcms/colors/editor/scmeditordialog.ui<br />
kcms/colors/editor/scmeditoreffects.cpp<br />
kcms/colors/editor/scmeditoreffects.h<br />
kcms/colors/editor/scmeditoreffects.ui<br />
kcms/colors/editor/scmeditoroptions.cpp<br />
kcms/colors/editor/scmeditoroptions.h<br />
kcms/colors/editor/scmeditoroptions.ui<br />
kcms/colors/editor/setpreview.ui<br />
kcms/colors/editor/setpreviewwidget.cpp<br />
kcms/colors/editor/setpreviewwidget.h<br />
kcms/colors/kcm_colors.desktop<br />
kcms/colors/kcolorschemeeditor.cpp<br />
kcms/colors/org.kde.kcolorschemeeditor.desktop<br />
kcms/colors/package/contents/ui/main.qml<br />
kcms/colors/package/metadata.desktop<br />
kcms/colors/preview.ui<br />
kcms/colors/previewwidget.cpp<br />
kcms/colors/previewwidget.h<br />
kcms/colors/scmeditorcolors.cpp<br />
kcms/colors/scmeditorcolors.h<br />
kcms/colors/scmeditorcolors.ui<br />
kcms/colors/scmeditordialog.cpp<br />
kcms/colors/scmeditordialog.h<br />
kcms/colors/scmeditordialog.ui<br />
kcms/colors/scmeditoreffects.cpp<br />
kcms/colors/scmeditoreffects.h<br />
kcms/colors/scmeditoreffects.ui<br />
kcms/colors/scmeditoroptions.cpp<br />
kcms/colors/scmeditoroptions.h<br />
kcms/colors/scmeditoroptions.ui<br />
kcms/colors/setpreview.ui<br />
kcms/colors/setpreviewwidget.cpp<br />
kcms/colors/setpreviewwidget.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>GB_2, nicolasfella, mart, abetts, ngraham, davidedmundson, plasma-devel, jraleigh, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol<br /></div>