<table><tr><td style="">leinir 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/D20986">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/D20986#460112" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20986#460112</a>, <a href="https://phabricator.kde.org/p/broulik/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@broulik</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>-1</p>

<p>Most KCMs actually use <tt style="background: #ebebeb; font-size: 13px;">onClicked</tt> to update the selected plugin and other bits. They don't actually set <tt style="background: #ebebeb; font-size: 13px;">currentIndex</tt> on the <tt style="background: #ebebeb; font-size: 13px;">ListView</tt> directly but have it updated based on whatever current scheme or theme etc is selected.</p>

<p>Can it be fixed such that the <tt style="background: #ebebeb; font-size: 13px;">onClicked</tt> isn't fired when the action is invoked? From what I can see the actions don't rely on <tt style="background: #ebebeb; font-size: 13px;">currentIndex</tt></p></div>
</blockquote>

<p>Unless i'm misunderstanding you, that's precisely what this code does - onClicked still gets triggered when clicking the thumbnail itself, this patch only affects the action buttons. The previous code was specifically added to set the current index, according to Nate in <a href="https://phabricator.kde.org/D18649#459586" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D18649#459586</a> :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R296 KDeclarative</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20986">https://phabricator.kde.org/D20986</a></div></div><br /><div><strong>To: </strong>leinir, ngraham<br /><strong>Cc: </strong>broulik, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>