<table><tr><td style="">davidhurka edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-dxqrzrzk3jpzthz/">(Show Details)</a><br />davidhurka edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-k4c4pz6egoialmh/">(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/D21195">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>There <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">is an action to toggle Change C</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">are menu items for every c</span>olor<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">s on and off</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> mode</span>, and <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">an action group to choose the Change</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">one {nav Normal</span> Colors<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> mode</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">}</span>.<br />
The menu is an <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">K</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Toggle</span>ActionMenu, so it can be plugged into the toolbar for quicker access.<br />
The <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">KActionMenu is checkable</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">toolbar button always shows the last used color mode</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">so toggling Change Colors just means to click the toolbar button once</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">so it can be enabled or disabled with one click</span>.<div style="padding: 8px 0;">...</div>This also fixes 407217, because the button does toggle now<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">. Additionally, every color mode can get a shortcut</span>.<div style="padding: 8px 0;">...</div>Screenshot after<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> (Left with {nav Invert Colors} and menubar menu, right with {nav Normal Colors}, and with/out toolbar button menu)</span>:<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">{F6821937}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6917434}</span><br />
<br />
TODO<div style="padding: 8px 0;">...</div>[<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">x</span>] Remove CheckableActionMenu by just adding the KActionMenu indepently to the KActionCollection, like done with the ‘Selection Tools’ button? See D21195#468175 (replaced by following)<br />
[<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">x</span>] Use ToggleActionMenu instead of CheckableActionMenu.<br />
[<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">x</span>] Show the current color mode in the toolbar button.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>Then, plug the Change Colors menu into the toolbar:<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Click it -> toggles</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Shows last used color mode.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Open</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Click</span> it -> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">choose another Change Colors mode</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">toggles</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Then, assign shortcuts</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Open its menu -> choose another color mode</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Shortcuts shouldn’t be removed after using them</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Then, assign shortcuts</span><br />
* Triggering <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">a</span> shortcut <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">for e. g. Change Paper Color also can toggle Change Colors on and off</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">updates the toolbar button.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21195">https://phabricator.kde.org/D21195</a></div></div><br /><div><strong>To: </strong>davidhurka, Okular, VDG<br /><strong>Cc: </strong>GB_2, davidre, aacid, ngraham, okular-devel, fbampaloukas, joaonetto, tfella, darcyshen<br /></div>