<table><tr><td style="">leinir updated this revision to Diff 55403.<br />leinir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-k6whtwqzwqh4jud/">(Show Details)</a><br />leinir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-iurnqok6ig7xxep/">(Show Details)</a><br />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/D20198">View Revision</a></tr></table><br /><div><div><p>Less invasive change, but with more (what seem like fairly reasonable) assumptions. We now operate explicitly on installed and uninstalled files as presented by KNS, rather than simply rechecking the theme locations (which it turns out is surprisingly intensive, but then again, it /is/ a disk IO operation, so perhaps not really that surprising).</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">When installing a new cursor theme <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">from KNewStuff, the current theme would become deselected due to a full refresh of the theme list.<br />
<br />
This also introduces a simple convenience helper originally from Discover, which simply runs a function based on a signal being fired, and then deletes itself. This might be useful elsewhere as well</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">(or ditto uninstalling) from KNewStuff</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but until the need arises it seems reasonable to leave it here</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the current theme would become deselected due to a full refresh of the theme list</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Install a theme using Get Hot New Stuff, see that after the list refreshes, the current theme is reselected<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">. Also uninstall a KNS originated theme and see the same is true</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/D20198?vs=55330&id=55403">https://phabricator.kde.org/D20198?vs=55330&id=55403</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20198">https://phabricator.kde.org/D20198</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/cursortheme/kcmcursortheme.cpp</div></div></div><br /><div><strong>To: </strong>leinir, Plasma, ngraham, broulik<br /><strong>Cc: </strong>apol, plasma-devel, Plasma, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart<br /></div>