<table><tr><td style="">romangg 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/D10855">View Revision</a></tr></table><br /><div><div><p>Why is the press and hold timer still needed? It should be enough to only test on double click expiration with this patch and open the menu on release of mouse button afterwards. We should accept this behavioral change in order to reduce code complexity (people should automatically understand that the menu opens from now on on button release after they have experienced it once trying it with long click). But if you want to split this up in a second patch I'm fine with it.</p>

<p>Otherwise I'm in favor of the change.</p>

<p>So do you want to change this diff or put it in a second diff?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R129 Window Decoration Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10855">https://phabricator.kde.org/D10855</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, graesslin, VDG<br /><strong>Cc: </strong>romangg, mart, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>