> 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? > > Harshit Agarwal wrote: > 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?
Hmm then it might just be because of something else I'm working on. I'll wait for someone else to try this out. - Shantanu ----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
