----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110498/#review32743 -----------------------------------------------------------
This looks fine, although I suppose that pressing F5 (common shortcut for refresh) when the File View is focused has no effect? If not, please implement it and update this patch, it may be needed to override keyPressEvent() of the view. - Matěj Laitl On May 18, 2013, 1:23 p.m., Abhinandan Ramaprasath wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110498/ > ----------------------------------------------------------- > > (Updated May 18, 2013, 1:23 p.m.) > > > Review request for Amarok. > > > Description > ------- > > A tiny patch that adds a refresh button to the file browser. > > > This addresses bug 213666. > https://bugs.kde.org/show_bug.cgi?id=213666 > > > Diffs > ----- > > src/browsers/filebrowser/FileBrowser.h 99b22c7 > src/browsers/filebrowser/FileBrowser.cpp 4b80774 > src/browsers/filebrowser/FileBrowser_p.h 39fa86d > > Diff: http://git.reviewboard.kde.org/r/110498/diff/ > > > Testing > ------- > > yes, works. > > > Thanks, > > Abhinandan Ramaprasath > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
