<table><tr><td style="">leinir created this revision.<br />leinir added a project: Kirigami.<br />Restricted Application added a subscriber: plasma-devel.
</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/D7814" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This ensures we can have actions on multiple pages which use the same shortcut keysequences without running into clashes (which then in turn disables the shortcuts for /any/ action using it, not only subsequent ones). If the action is not in a page (that is, the default state for an action), it will simply always have the keysequence assigned. This means that, should you wish, you can manually handle this behaviour, which is the current situation.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Without this patch, having two pages on the page stack at the same time using the same shortcut keysequence for their actions (for example, two instances of the same page), will result in all those actions failing to work once the second page is instantiated.</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/D7814" rel="noreferrer">https://phabricator.kde.org/D7814</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/controls/Action.qml<br />
src/controls/Page.qml<br />
src/controls/private/PageActionPropertyGroup.qml</div></div></div><br /><div><strong>To: </strong>leinir, Kirigami, mart<br /><strong>Cc: </strong>plasma-devel, Kirigami, apol, mart, hein<br /></div>