<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/111047/">http://git.reviewboard.kde.org/r/111047/</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;">I found a small glitch, please reproduce this error.
Steps:
1) Go to Browse Pics/Videos/Music
2) Play a media ( eg pictures will open in the slideshow)
3) Pressing Escape key once will take you one level back as intended, but again pressing escape key breaks the flow now.Nothing happens, and that is intuitively due to currentBrowsingBackend loosing track. In short, please try pressing "Escape Key" twice in the slideshow to go back to the homescreen. It wont work.
OR
for that matter, even when you go to Browse Pics/Videos/Music, in the media browser itself, try pressing Escape Key to go back again to the homescreen, the PMC breaks in this instance.</pre>
<br />
<p>- Akshay</p>
<br />
<p>On June 16th, 2013, 7:36 a.m. UTC, Shantanu Tushar wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma, Akshay Ratan and Sinny Kumari.</div>
<div>By Shantanu Tushar.</div>
<p style="color: grey;"><i>Updated June 16, 2013, 7:36 a.m.</i></p>
<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;">At places we assume that there always be a valid browsing backend. In many situations this isn't true, which is exactly what this patch handles.</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;">Tested some scenarios that I could think of. Please test few scenarios and see if this works.
Akshay: I think the new if conditions make previousBackend necessary, please check that your feature still works fine.</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>mediaelements/mediabrowser/MediaBrowser.qml <span style="color: grey">(e7aa109)</span></li>
<li>shells/newshell/package/contents/ui/mediacenter.qml <span style="color: grey">(d533b60)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111047/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>