<table><tr><td style="">ouwerkerk 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/D26354">View Revision</a></tr></table><br /><div><div><p>Are we certain about naming here: <tt style="background: #ebebeb; font-size: 13px;">SwipeAction.isDelete</tt>? Maybe <tt style="background: #ebebeb; font-size: 13px;">SwipeAction.remove</tt> or <tt style="background: #ebebeb; font-size: 13px;">SwipeAction.removeFromList</tt> ? It doesn't necessarily have to be a real "delete" action that is backing this, maybe all you want to convey with the name of this setting is that the list entry will be removed from the UI if enabled?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26354">https://phabricator.kde.org/D26354</a></div></div><br /><div><strong>To: </strong>cblack, VDG, Kirigami<br /><strong>Cc: </strong>ouwerkerk, ngraham, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, apol, ahiemstra, davidedmundson, mart, hein<br /></div>