<table><tr><td style="">ndavis added a comment.
</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/D25814">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25814#574129" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25814#574129</a>, <a href="https://phabricator.kde.org/p/hpereiradacosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hpereiradacosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Few more comments on this:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">general: you will never be able to make all the opiniated people happy, and you have to draw a line (otherwise your code will become bloated, buggy, and unmaintainable)</li>
<li class="remarkup-list-item">regarding this specific case: many widget style will not implement this new color. For those this will just generate bugs reports: why is my color scheme not respectd ?</li>
<li class="remarkup-list-item">some widget styles (oxygen at least, but I'm sure there are others) use two colors for frames and separators, to mimic shadows or relief effects. Adding one single color for this will not fit such schemes.</li>
<li class="remarkup-list-item">in the end if you need an extra color for a given theme (be it future-breeze or whatever), there is also the possibility to add it as a extra option for this specific theme, rather than forcing it to kcolorscheme and imposing it to all styles (or making kcolorscheme broken for all the styles that wont use it). This is how window decoration shadows and glow were handled to oxygen. I think this change will break more things than it will fix, especially if the fix is to make some opinionated people happy.</li>
</ul></div>
</blockquote>



<ul class="remarkup-list">
<li class="remarkup-list-item">I know it's not a great reason, but I want the setting too. I'm not only making it for others.</li>
<li class="remarkup-list-item">The most popular 3rd party themes (Kvantum themes) don't even support colorschemes. For Kvantum users, colorschemes are only useful for setting the colors on QML apps, which Kvantum doesn't properly set the colors for. AFAIK, the only other commonly used QStyles are Breeze, Oxygen and QStyles provided by Qt. Qt's QStyles only uses colors that map to QPalette colors. The only customizable colors which don't map to QPalette that Oxygen and Breeze use are View FocusColor, View HoverColor and View NegativeText. Oxygen also uses KColorScheme's shade colors. If KColorScheme is only used by us, what is is the point of having KColorScheme if we barely use it and we cannot extend it for use with our own software?</li>
<li class="remarkup-list-item">I don't intend to change Oxygen.</li>
<li class="remarkup-list-item">The VDG has been trying to make it easier to navigate through SySe, so I want to avoid putting more settings in poorly visible locations. I also want access to this color outside of the Breeze widget style so that it can be used in Plasmashell and QML apps. I suppose I wouldn't have to make this customizable for users to do the last part.</li>
<li class="remarkup-list-item">I'm not sure what you mean by "break". Do you mean themes that previously worked will stop working?</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R265 KConfigWidgets</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25814">https://phabricator.kde.org/D25814</a></div></div><br /><div><strong>To: </strong>ndavis, Frameworks, VDG<br /><strong>Cc: </strong>davidedmundson, filipf, cfeck, hpereiradacosta, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>