<table><tr><td style="">ngraham added a subscriber: GB_2.<br />ngraham 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/D27540">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/D27540#652692" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D27540#652692</a>, <a href="https://phabricator.kde.org/p/ervin/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ervin</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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/D27540#652664" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D27540#652664</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>David asked for VDG to approve before this landed, which wasn't done.</p></div>
</blockquote>
<p>Dude, I jumped through all the hoops for the past weeks. Also it got no further reply after I updated the screenshot almost two weeks ago so yes I assumed you guys had nothing else.</p></div>
</blockquote>
<p>Sorry I didn't have the time to test it again. It's not just about the UI design itself, but also avoid obvious visual glitches, like this: <a href="https://phabricator.kde.org/F8249569" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8249569: Screenshot_20200420_104632.png</a></p>
<p>There are also numerous cases in other KCMs where the indicator gets cut off, positioned strangely, or isn't visible at all for a non-default setting:<br />
<a href="https://phabricator.kde.org/F8249574" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8249574: Screenshot_20200420_105414.png</a><br />
<a href="https://phabricator.kde.org/F8249578" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8249578: Screenshot_20200420_105450.png</a><br />
<a href="https://phabricator.kde.org/F8249582" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8249582: Peek 2020-04-20 10-57.webm</a></p>
<p>How are we going to fix that? The diversity of user interfaces we have throughout KDE software makes me skeptical that any kind of auto-generated icon placement can ever work, let alone look good. Where will the icons go in list items? Grid views? And so on. To be completely honest, if nobody could come up with a good UI, it might be a sign that the feature itself needs to be re-thought. I remain unconvinced that this is the best way to show that there are changed settings. I think <a href="https://phabricator.kde.org/p/GB_2/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@GB_2</a>'s idea of displaying the original or previous state when hovering over the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Defaults</span></span></span> or <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Reset</span></span></span> button made more sense. As is, I don't really understand who the target user is for this feature.</p>
<p>IMO this feature would have benefited from being outlined first in a Phabricator task and soliciting <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">#VDG</a> feedback before coding began, so we didn't frustratingly go back and forth in the patches. Given the above visual regressions and broken behavior on multiple KCMs, I think this needs to be reverted and discussed and re-thought in a central location. Sorry. :(</p></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/D27540">https://phabricator.kde.org/D27540</a></div></div><br /><div><strong>To: </strong>ervin, ngraham, davidedmundson, meven, crossi, bport, VDG, ndavis, broulik<br /><strong>Cc: </strong>GB_2, alexde, ndavis, iasensio, davidre, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>