----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126295/ -----------------------------------------------------------
(Updated Dec. 14, 2015, 11:31 a.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks, David Faure and Emmanuel Pescosta. Changes ------- Submitted with commit d34b37b6831429f7a48edec7de77cb88524b3784 by Weng Xuetian to branch master. 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