<table><tr><td style="">hein 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/D13192">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Could a different approach work here? No change on hover, but just on click? The user controls the entire action.</p></blockquote>

<p>Can you explain what you dislike about my patch enough that makes you think an entirely different approach is in order, or why you think it fails to solve the problem? It's a bit weird to react to a patch with "lol ... ok, maybe something else entirely" in review without that buildup.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13192">https://phabricator.kde.org/D13192</a></div></div><br /><div><strong>To: </strong>hein, ngraham, davidedmundson, rkflx, cfeck<br /><strong>Cc: </strong>mart, abetts, zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol<br /></div>