<table><tr><td style="">wbauer created this revision.<br />wbauer added a reviewer: Amarok.<br />wbauer added a project: Amarok.<br />Herald removed a project: Amarok.<br />wbauer 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/D24381">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Commit <a href="https://phabricator.kde.org/R181:0b2b243b2b70471c32d282157624379da5214051" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">0b2b243b</a> removed the call to <tt style="background: #ebebeb; font-size: 13px;">KGlobalAccel::setGlobalShortcut()</tt> because it registered <tt style="background: #ebebeb; font-size: 13px;">Space</tt> as global shortcut, which of course is a bad idea.<br />
But that made it impossible to set up a global shortcut at all (and made an existing one have no effect).</p>

<p>To fix this, add it back and register an empty global shortcut, like the KDE4 code does.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>A global shortcut for Play/Pause can again be set in systemsettings5->Shortcuts->Global Shortcuts (it doesn't have a key assigned by default), and pressing the key does work as expected afterwards even if Amarok's window is not active.</p>

<p>The local shortcut (<tt style="background: #ebebeb; font-size: 13px;">Space</tt> by default) still works as well.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R181 Amarok</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24381">https://phabricator.kde.org/D24381</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/ActionClasses.cpp</div></div></div><br /><div><strong>To: </strong>wbauer, Amarok<br /><strong>Cc: </strong>amarok-devel, Amarok<br /></div>