From 584c8b2ba78071a805c0960841c636a331671ab3 Mon Sep 17 00:00:00 2001
From: Andrea Diamantini <adjam7@gmail.com>
Date: Sun, 29 Aug 2010 00:03:54 +0200
Subject: First fixes, 4 suggestions

---
 src/urlbar/completionwidget.cpp | 36 ++++++++++++++++++++----------------
 src/urlbar/urlresolver.cpp      |  6 ++++--
 2 files changed, 24 insertions(+), 18 deletions(-)

(limited to 'src')

diff --git a/src/urlbar/completionwidget.cpp b/src/urlbar/completionwidget.cpp
index c1f5d83b..bd907fa9 100644
--- a/src/urlbar/completionwidget.cpp
+++ b/src/urlbar/completionwidget.cpp
@@ -79,6 +79,8 @@ void CompletionWidget::insertSearchList(const UrlSearchList &list, const QString
     int i = 0;
     foreach(const UrlSearchItem &item, _list)
     {
+        kDebug() << "ITEM URL: " << item.url;
+        kDebug() << "ITEM TYPE: " << item.type;
         ListItem *suggestion = ListItemFactory::create(item, text, this);
         suggestion->setBackgroundRole(i % 2 ? QPalette::AlternateBase : QPalette::Base);
         connect(suggestion, 
@@ -95,28 +97,29 @@ void CompletionWidget::insertSearchList(const UrlSearchList &list, const QString
 
 void CompletionWidget::updateSearchList(const UrlSearchList &list, const QString& text)
 {
-    kDebug() << "TYPED STRING: " << _typedString;
-    kDebug() << "text from suggestion: " << text;
-    
     if(_isSuggesting || !isVisible() || _typedString != text)
         return;
     
     _isSuggesting = true;
+
+    kDebug() << "LIST COUNT: " << list.count();
+    UrlSearchList sugList = list.mid(0,4);
+    kDebug() << "SUGLIST COUNT: " << sugList.count();
     
-    // clean up eventual old suggestions
-    if(_suggestionsNumber > 0)
-    {
-        int offset = _list.count();
-        for(int i = offset; i < offset + _suggestionsNumber; ++i)
-        {
-            QLayoutItem *item = layout()->takeAt( i );
-            delete item;
-        }
-    }
+//     // clean up eventual old suggestions
+//     if(_suggestionsNumber > 0)
+//     {
+//         int offset = _list.count();
+//         for(int i = offset; i < offset + _suggestionsNumber; ++i)
+//         {
+//             QLayoutItem *item = layout()->takeAt( i );
+//             delete item;
+//         }
+//     }
 
     // add new suggestions to the list
     int i = _list.count();
-    Q_FOREACH(const UrlSearchItem &item, list)
+    Q_FOREACH(const UrlSearchItem &item, sugList)
     {
         ListItem *suggestion = ListItemFactory::create(item, text, this);
         suggestion->setBackgroundRole(i % 2 ? QPalette::AlternateBase : QPalette::Base);
@@ -129,8 +132,8 @@ void CompletionWidget::updateSearchList(const UrlSearchList &list, const QString
         suggestion->setObjectName(QString::number(i++));
         layout()->addWidget(suggestion);
     }
-    _suggestionsNumber = list.count();
-    _list.append(list);
+    _suggestionsNumber = sugList.count();
+    _list.append(sugList);
     sizeAndPosition();
     _isSuggesting = false;
 }
@@ -218,6 +221,7 @@ void CompletionWidget::clear()
         delete child;
     }
     _currentIndex = 0;
+//     _suggestionsNumber = 0;
 }
 
 
diff --git a/src/urlbar/urlresolver.cpp b/src/urlbar/urlresolver.cpp
index 300f33ef..985b15c3 100644
--- a/src/urlbar/urlresolver.cpp
+++ b/src/urlbar/urlresolver.cpp
@@ -133,10 +133,10 @@ UrlSearchList UrlResolver::orderedSearchItems()
 
     //compute lists
     computeSuggestions();
+    computeHistory();
     computeQurlFromUserInput();
     computeWebSearches();
     computeBookmarks();
-    computeHistory();
 
     return orderLists();
 }
@@ -263,6 +263,8 @@ UrlSearchList UrlResolver::orderLists()
     historyCount = _history.count();
     bookmarksCount = _bookmarks.count();
     commonCount = common.count();
+    
+    kDebug() << "HISTORY COUNT: " << historyCount;
 
     //now fill the list to MAX_ELEMENTS
     if(availableEntries > 0)
@@ -352,7 +354,7 @@ void UrlResolver::computeHistory()
 void UrlResolver::computeBookmarks()
 {
     QList<KBookmark> found = Application::bookmarkProvider()->find(_typedString);
-
+    kDebug() << "FOUND: " << found.count();
     Q_FOREACH(const KBookmark &b, found)
     {
         UrlSearchItem gItem(UrlSearchItem::Bookmark, b.url().url(), b.fullText());
-- 
cgit v1.2.1