Review Request 116063: Implementation of the backspace key as an option to go back to the previous window.

[Harshit Agarwal]

> On Feb. 26, 2014, 2:52 p.m., Shantanu Tushar wrote:
> > I see a weird behavior with this patch, if you follow these steps-
> > 1. Open PMC, select "All Videos"
> > 2. Press backspace to go back to home screen
> > 3. Press enter to go to "All Videos" again
> > 
> > This leads to a blank browser without anything in it. Can you verify this and fix?

I seem to be unable to reproduce the error. On doing the exact same steps, it goes to the 'All Videos' window, and shows the videos listed there, with default focus at the search bar. To use the backspace functionality again, one would have to press the down arrow key once, in order to lose focus on the search bar and gain focus on the videos over there, then the backspace would work to go back to the previous window.

Can you explain on the problem bit more so that it can be reproduced?

However I noticed a thing, when PMC is opened for the first time, and 'All Videos' is selected, it takes around 2 seconds to load the videos with their thumbnails. Probably this is the time taken for it to read the folder, and then display the content. Or any anomaly here?


- Harshit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116063/#review50939
-----------------------------------------------------------


On Feb. 26, 2014, 7:45 a.m., Harshit Agarwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116063/
> -----------------------------------------------------------
> 
> (Updated Feb. 26, 2014, 7:45 a.m.)
> 
> 
> Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> Through this patch, the user can go to the previous window through the backspace key also. The issue with the focus on the Search Bar has been solved. The backspace option works only when the search bar is not in focus.
> 
> 
> Diffs
> -----
> 
>   shells/newshell/package/contents/ui/mediacenter.qml 49ea403 
> 
> Diff: https://git.reviewboard.kde.org/r/116063/diff/
> 
> 
> Testing
> -------
> 
> It has been tested.
> 
> 
> Thanks,
> 
> Harshit Agarwal
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140226/bd5b1867/attachment.html>