From c07a97c4be1b5118026ea95831fd1edf7fac34e9 Mon Sep 17 00:00:00 2001
From: Emmanuel Pescosta <emmanuelpescosta099@gmail.com>
Date: Tue, 21 Oct 2014 22:54:15 +0200
Subject: [PATCH] Use the path instead of the display string in
 PlacesItemModel::closestItem, fixes the determination of the closest item for
 me.

---
 src/panels/places/placesitemmodel.cpp | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/panels/places/placesitemmodel.cpp b/src/panels/places/placesitemmodel.cpp
index e49a6a9da6..8e6494919e 100644
--- a/src/panels/places/placesitemmodel.cpp
+++ b/src/panels/places/placesitemmodel.cpp
@@ -214,8 +214,12 @@ int PlacesItemModel::closestItem(const QUrl& url) const
 
     for (int i = 0; i < count(); ++i) {
         const QUrl itemUrl = placesItem(i)->url();
-        if (itemUrl.isParentOf(url)) {
-            const int length = itemUrl.toDisplayString().length();
+        if (url == itemUrl) {
+            // We can't find a closer one, so stop here.
+            foundIndex = i;
+            break;
+        } else if (itemUrl.isParentOf(url)) {
+            const int length = itemUrl.path().length();
             if (length > maxLength) {
                 foundIndex = i;
                 maxLength = length;