----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125988/#review88860 -----------------------------------------------------------
src/widgets/kurlcompletion.cpp (line 531) <https://git.reviewboard.kde.org/r/125988/#comment60885> are you sure this hunk is necessary? QUrl has no special handling of "." in paths. So m_fileName = m_kurl.fileName() would also just set it to ".", like your special code does. (and if that's right then I don't see the purpose of the two new vars...) Wasn't only the last hunk of the patch that really made a difference? - David Faure On Nov. 9, 2015, 8:30 a.m., Emmanuel Pescosta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125988/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2015, 8:30 a.m.) > > > Review request for KDE Frameworks and David Faure. > > > Bugs: 354981 > https://bugs.kde.org/show_bug.cgi?id=354981 > > > Repository: kio > > > Description > ------- > > Currently KUrlCompletion only offers autocompletion for hidden folders when > you input at least one additional character after the dot. > With this patch all hidden folders will be offered when only a dot is present. > > This behaviour is test covered. > > > Diffs > ----- > > autotests/kurlcompletiontest.cpp ca8563c > src/widgets/kurlcompletion.cpp c6764e4 > > Diff: https://git.reviewboard.kde.org/r/125988/diff/ > > > Testing > ------- > > All test cases for KUrlCompletion pass > > > Thanks, > > Emmanuel Pescosta > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
