<table><tr><td style="">leinir retitled this revision from "Disable shortcuts for actions not on the current page" to "Disable shortcuts when an action is disabled".<br />leinir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-efgsf3shviyvael/" rel="noreferrer">(Show Details)</a><br />leinir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-totxbsgp2ctkqon/" rel="noreferrer">(Show Details)</a>
</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>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This e<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">nsures 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).</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">qualises the API with QQC2's actions (which also disable shortcuts when the action itself is disabled)<br />
<br />
nb: This pushes the task of disabling actions on sub-pages and anywhere else not currently supposed to be active onto the app developer,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">If the action is not in a page (that is</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but it reduces complexity a very great deal</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the default state for an action),</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">and also means when</span> it <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">will simply always have the keysequence assigned</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">comes time to switch to QQC2.Action</span>. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">This means that</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">It is less magic</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">should you wish,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but this is not necessarily a bad thing.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">you can manually handle this behaviour</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">It would still be nice to have everything on non-current pages entirely disabled when the page is not current (or indeed visible)</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">which is the current situation</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but this works</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Without this patch,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Previously, disabling an action would still cause the shortcut sequence defined for that action to be active, which very easily caused clashes and required manual intervention.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">having two pages on the page stack at the same time using the same shortcut keysequence for their actions (for example</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Now</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">two instances of the same page)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">simply disabling the action will propagate that to the shortcut</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">will </span>result<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> in all those actions failing to work once the second page is instantiated</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ing in simpler code than the previous state</span>.</div></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>To: </strong>leinir, Kirigami, mart<br /><strong>Cc: </strong>plasma-devel, Kirigami, apol, mart, hein<br /></div>