----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126295/#review89378 -----------------------------------------------------------
Sorry for breaking this code :/ Would it be possible to add some test cases to prevent such problems in future? - Emmanuel Pescosta On Dec. 12, 2015, 10:35 a.m., Xuetian Weng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126295/ > ----------------------------------------------------------- > > (Updated Dec. 12, 2015, 10:35 a.m.) > > > Review request for KDE Frameworks, David Faure and Emmanuel Pescosta. > > > Repository: kio > > > Description > ------- > > 9dbe36f734b5b839b2a6a934fad29d639e954498 breaks some assumption on > startIndex. The argument index passed to buttonUrl is based number of '/'in > full url. Because of that, url like "sftp://a...@b.com/a/b"; is now shown as > "sftp:a...@b.com" / "b" / "b" in dolphin url bar. > > This patch changes all relevant code that calls buttonUrl() to use url.path() > instead of url.toDisplayString() to count the number of "/". > > > Diffs > ----- > > src/filewidgets/kurlnavigator.cpp 848e98b > > Diff: https://git.reviewboard.kde.org/r/126295/diff/ > > > Testing > ------- > > Remote url -> ok > Local url -> ok > Remote url in places and try browse some sub folder -> ok > Local url in places and try browse some sub folder -> ok > > > Thanks, > > Xuetian Weng > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
