----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126295/ -----------------------------------------------------------
(Updated Dec. 12, 2015, 9: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 (updated) ----- 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
