<table><tr><td style="">ahiemstra 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/D24634">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>now has the order main action, left, right. before was left, main right.<br />
 it does change a bit the behavior, tough i see the point of making sure that the main action is always the last being hidden (i wonder how it would be making it hide last even if not the last)</p></blockquote>

<p>While working on it, I realised that the left, main, right order actually doesn't make sense when placed in the toolbar, considering the main action is the most important action and normal order would be left-to-right with the most imporant on the leftmost place. It is easy enough to change back but I do consider this more "correct".</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>testing it i did find a major problem: if i resize the window quickly (kirigami gallery 2 columns mode) when it goes from icon+text to just icon, the window freezes for an instant, seems quite expensive, while before didn't do that<br />
 from the code nothing should have gotten particularly worse, so not sure how to debug</p></blockquote>

<p>The main cause for this I believe is in the toolbar header file. It currently removes the overflow actions from the main action list when switching to the icon-only layout, which causes removal/recreation of hidden actions used for layout. I've been looking at implementing this behaviour in the actual actiontoolbar, which hopefully removes the need for all of that and speed up things. It's why this is still WIP. :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>contextactions_in_toolbar</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24634">https://phabricator.kde.org/D24634</a></div></div><br /><div><strong>To: </strong>ahiemstra, Kirigami, ngraham<br /><strong>Cc: </strong>mart, ngraham, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, apol, ahiemstra, davidedmundson, hein<br /></div>