Change Details

I added a top separator for QML applet configuration windows in D20908 This separator is the equivalent of KWin's separator, minus the highlight color. I have been regretting the decision to draw it ever since the patch was comitted and believe it to be wrong for several reasons: - it's an infrigment on user choice: it doesn't respect the fact the user has turned off KWin's titlebar separator - it's inconsistent with qwidgets: there is no top separator there - it's degrades the goal of trying to have a uniform appearance between the window manager and window in the case of Breeze Light and Breeze Dark because it distinctly points out the titlbar - it's superfluous (not visible) when used with the default color scheme anyway With this patch only the sidebar is enclosed, which solves all of the points above with the exception of the second one (which just warrants this patch be replicated for qwidgets)

I added a top separator for QML applet configuration windows in D20908 This separator is the equivalent of KWin's separator, minus the highlight color. I have been regretting the decision to draw it ever since the patch was committed and believe it to be wrong for several reasons: - it's an infrigment on user choice: it doesn't respect the fact the user has turned off KWin's titlebar separator - it's inconsistent with qwidgets: there is no top separator there - it's degrades the goal of trying to have a uniform appearance between the window manager and window in the case of Breeze Light and Breeze Dark because it distinctly points out the titlebar - it's superfluous (not visible) when used with the default color scheme anyway This patch therefore removes the separator. For consistency reasons the separator will be drawn by the tools area code provided the option is turned on.

I added a top separator for QML applet configuration windows in D20908 This separator is the equivalent of KWin's separator, minus the highlight color. I have been regretting the decision to draw it ever since the patch was committed and believe it to be wrong for several reasons: - it's an infrigment on user choice: it doesn't respect the fact the user has turned off KWin's titlebar separator - it's inconsistent with qwidgets: there is no top separator there - it's degrades the goal of trying to have a uniform appearance between the window manager and window in the case of Breeze Light and Breeze Dark because it distinctly points out the titlebar - it's superfluous (not visible) when used with the default color scheme anyway With tThis patch only the sidebar is enclosed,therefore removes the separator. which solves all of the points above with the exception of the second one (which just warrants this patch be replicated for qwidgets)For consistency reasons the separator will be drawn by the tools area code provided the option is turned on.