<div>broulik created this revision.<br />
broulik added reviewers: Plasma, hein.<br />
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.<br />
broulik added a project: Plasma.<br />
Herald added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>This removes the custom RubberBand that is painted using QStyle and instead uses a traditional Rectangle.</p>
<p>We don't use "Desktop Widget Style" in Plasma anyway. The rubber band now follows the theme's highlight color and has a nice fade animation when it disappears.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Rubber band selection works fine.<br />
Repeated quick moving and selecting doesn't make it go nuts, even when the fade out animation is still running.<br />
Auto scroll in both popup and as containment is erratic; not sure if this is caused by this patch, however.</p>
<p>Using Tilain the rubber band is now white, for example, using Oxygen it's a fainter blue.</p>
<p>I did not yet remove the RubberBand cpp stuff as I wanted to get some feedback first before messing up my local repository :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1028" rel="noreferrer">https://phabricator.kde.org/D1028</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderView.qml</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein<br /><strong>Cc: </strong>plasma-devel<br /></div>