<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124773/">https://git.reviewboard.kde.org/r/124773/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Sorry, but that is not acceptable that way, you are using shortcuts that are already defined on the system, like Ctrl+A, this can't be used. (FWIW: Ctrl+A is used on pretty much every system I know to select everything).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So please check again, do NOT use Standard shortcuts or already defined global ones, those already in use can be found quite easily in the systemsettings for KDE.</p></pre>
<br />
<p>- Myriam Schweingruber</p>
<br />
<p>On August 17th, 2015, 9:26 a.m. CEST, Aditya Dev Sharma wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Amarok.</div>
<div>By Aditya Dev Sharma.</div>
<p style="color: grey;"><i>Updated Aug. 17, 2015, 9:26 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=300979">300979</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
amarok
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">JJ : Bug - 300979 : Patch to add some shortcuts for various actions in createActions() in MainWindow</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Built and tested.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Initially added some more shortcuts but couldn't test them. Will have to go through the Amarok User Hand Book to see how are things supposed to go. So will update in a day or two.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Action : Shortcut --------> SLOT</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Clear Playlist : CTRL + SHIFT + C ---------------> SLOT(clear() )
Add Stream : CTRL + A ---------------> SLOT(slotAddStream() )
Export Playlits : CTRL + SHIFT + E ---------------> SLOT(exportPlaylist() )
Bookmarks Manager : CTRL + B ---------------> SLOT(slotShowBookmarkManager() )
Equalizer : SHIFT + E ---------------> SLOT(slotShowEqualizer() )
Cover Manager : CTRL + ALT + C ---------------> SLOT(slotShowCoverManager() )
Synchronize Statistics : CTRL + S ---------------> SLOT(synchronize() )</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">/<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"> Rev 2 </em>/
Bookmark Media Sources View : SHIFT + B ----------> SLOT(bookmarkCurrentBrowserView() )
Bookmark Playlist Setup : ALT + B ---------------> bookmarkCurrentPlaylistView() )
Bookmark Context Applets : CTRL + SHIFT + B ------> bookmarkCurrentContextView() )
Disable Dynamic Playlist : ALT + SHIFT + D -------> connected inside dynamic playlist category</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">/<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"> Rev3 </em>/
About Amarok : CTRL + SHIFT + A -----------------> SLOT(showAbout())
Diagnostics : CTRL + ALT + D -----------------> SLOT(slotShowDiagnosticsDialog())
Report bug : ALT + SHIFT + R -----------------> SLOT(showReportBug())</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/MainWindow.cpp <span style="color: grey">(22fb8ce)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124773/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>