<table><tr><td style="">camiloh created this revision.<br />camiloh added a reviewer: Kirigami.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />camiloh requested review of this revision.
</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/D22974">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The ideal is to be able to add regular qqc2 actions to the actiontoolbar, pageheaders, globaldrawers and different components, and make kirigami.action based on qqc2.action, so implementations are not duplicated.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>So far i have tested this with different kirigami components, such as pageheadertoolbar, globaldrawers, kirigamiactiontoolbar, and it is working.<br />
There is an issue now with the PrivateActionToolButton, since K.Actions is based on QQC2.Action, it make sthe control to draw two icons, one from PrivateActionToolBar own contentItem implementation and another one from the style background</p>

<p>I will continue testing this, and pls let me know how i could proceed better with the PrivateActionToolButton issue.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>k-action-to-qqc2-action (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22974">https://phabricator.kde.org/D22974</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/controls/Action.qml<br />
src/controls/ActionToolBar.qml<br />
src/controls/GlobalDrawer.qml<br />
src/controls/private/GlobalDrawerActionItem.qml<br />
src/controls/private/PrivateActionToolButton.qml<br />
src/controls/private/globaltoolbar/ToolBarPageHeader.qml</div></div></div><br /><div><strong>To: </strong>camiloh, Kirigami<br /><strong>Cc: </strong>plasma-devel, fbampaloukas, domson, dkardarakos, apol, davidedmundson, mart, hein<br /></div>