diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/newtabpage.cpp | 63 | ||||
| -rw-r--r-- | src/newtabpage.h | 3 | ||||
| -rw-r--r-- | src/webpage.cpp | 13 | ||||
| -rw-r--r-- | src/webpage.h | 3 | 
4 files changed, 34 insertions, 48 deletions
| diff --git a/src/newtabpage.cpp b/src/newtabpage.cpp index e9aa0626..0372aad3 100644 --- a/src/newtabpage.cpp +++ b/src/newtabpage.cpp @@ -2,7 +2,7 @@  *  * This file is a part of the rekonq project  * -* Copyright (C) 2009-2011 by Andrea Diamantini <adjam7 at gmail dot com> +* Copyright (C) 2009-2012 by Andrea Diamantini <adjam7 at gmail dot com>  * Copyright (C) 2010 by Matthieu Gicquel <matgic78 at gmail dot com>  *  * @@ -300,14 +300,10 @@ void NewTabPage::favoritesPage()      for (int i = 0; i < urls.count() ; ++i)      {          KUrl url = KUrl(urls.at(i)); -        QWebElement prev; -        if (url.isEmpty()) -            prev = emptyPreview(i); -        else if (!WebSnap::existsImage(url)) -            prev = loadingPreview(i, url); -        else -            prev = validPreview(i, url, QString::number(i + 1) + " - " + names.at(i)); +        QWebElement prev = url.isEmpty() +                           ? emptyPreview(i) +                           : validPreview(i, url, QString::number(i + 1) + " - " + names.at(i));          m_root.appendInside(prev);      } @@ -426,9 +422,7 @@ void NewTabPage::closedTabsPage()          if (item.url.isEmpty())              continue; -        prev = WebSnap::existsImage(KUrl(item.url)) -               ? validPreview(i, item.url, item.title) -               : loadingPreview(i, item.url); +        prev = validPreview(i, item.url, item.title);          prev.setAttribute(QL1S("id"),  QL1S("preview") + QVariant(i).toString());          hideControls(prev); @@ -555,32 +549,37 @@ QWebElement NewTabPage::emptyPreview(int index)  } -QWebElement NewTabPage::loadingPreview(int index, const KUrl &url) -{ -    QWebElement prev = markup(QL1S(".thumbnail")); - -    prev.findFirst(QL1S(".preview img")).setAttribute(QL1S("src"), -            QL1S("file:///") + KStandardDirs::locate("appdata", "pics/busywidget.gif")); -    prev.findFirst(QL1S("span a")).setPlainText(i18n("Loading Preview...")); -    prev.findFirst(QL1S("a")).setAttribute(QL1S("href"), url.toMimeDataString()); - -    setupPreview(prev, index); -    showControls(prev); - -    // NOTE: we need the page frame for two reasons -    // 1) to link to the WebPage calling the snapFinished slot -    // 2) to "auto-destroy" snaps on tab closing :) -    QWebFrame *frame = qobject_cast<QWebFrame *>(parent()); -    WebSnap *snap = new WebSnap(url, frame); -    connect(snap, SIGNAL(snapDone(bool)), frame->page(), SLOT(updateImage(bool)), Qt::UniqueConnection); -    return prev; -} +// NOTE: comment this out WITHOUT really deleting. May be of inspiration... +// QWebElement NewTabPage::loadingPreview(int index, const KUrl &url) +// { +//     QWebElement prev = markup(QL1S(".thumbnail")); +// +//     prev.findFirst(QL1S(".preview img")).setAttribute(QL1S("src"), +//             QL1S("file:///") + KStandardDirs::locate("appdata", "pics/busywidget.gif")); +//     prev.findFirst(QL1S("span a")).setPlainText(i18n("Loading Preview...")); +//     prev.findFirst(QL1S("a")).setAttribute(QL1S("href"), url.toMimeDataString()); +// +//     setupPreview(prev, index); +//     showControls(prev); +// +//     // NOTE: we need the page frame for two reasons +//     // 1) to link to the WebPage calling the snapFinished slot +//     // 2) to "auto-destroy" snaps on tab closing :) +//     QWebFrame *frame = qobject_cast<QWebFrame *>(parent()); +//     WebSnap *snap = new WebSnap(url, frame); +//     connect(snap, SIGNAL(snapDone(bool)), frame->page(), SLOT(updateImage(bool)), Qt::UniqueConnection); +//     return prev; +// }  QWebElement NewTabPage::validPreview(int index, const KUrl &url, const QString &title)  {      QWebElement prev = markup(QL1S(".thumbnail")); -    QString previewPath = QL1S("file://") + WebSnap::imagePathFromUrl(url); + +    QString previewPath = WebSnap::existsImage(url) +                          ? QL1S("file://") + WebSnap::imagePathFromUrl(url) +                          : rApp->iconManager()->iconPathForUrl(url) +                          ;      prev.findFirst(QL1S(".preview img")).setAttribute(QL1S("src") , previewPath);      prev.findFirst(QL1S("a")).setAttribute(QL1S("href"), url.toMimeDataString()); diff --git a/src/newtabpage.h b/src/newtabpage.h index aae8da1e..40627c6c 100644 --- a/src/newtabpage.h +++ b/src/newtabpage.h @@ -2,7 +2,7 @@  *  * This file is a part of the rekonq project  * -* Copyright (C) 2009-2011 by Andrea Diamantini <adjam7 at gmail dot com> +* Copyright (C) 2009-2012 by Andrea Diamantini <adjam7 at gmail dot com>  * Copyright (C) 2010 by Matthieu Gicquel <matgic78 at gmail dot com>  *  * @@ -82,7 +82,6 @@ private:      // Previews handling      QWebElement emptyPreview(int index); -    QWebElement loadingPreview(int index, const KUrl &url);      QWebElement validPreview(int index, const KUrl &url, const QString &title);      QWebElement tabPreview(int winIndex, int tabIndex, const KUrl &url, const QString &title); diff --git a/src/webpage.cpp b/src/webpage.cpp index 61cc50d6..9dee0967 100644 --- a/src/webpage.cpp +++ b/src/webpage.cpp @@ -6,7 +6,7 @@  * Copyright (C) 2008 Dirk Mueller <mueller@kde.org>  * Copyright (C) 2008 Urs Wolfer <uwolfer @ kde.org>  * Copyright (C) 2008 Michael Howell <mhowell123@gmail.com> -* Copyright (C) 2008-2011 by Andrea Diamantini <adjam7 at gmail dot com> +* Copyright (C) 2008-2012 by Andrea Diamantini <adjam7 at gmail dot com>  * Copyright (C) 2010 by Matthieu Gicquel <matgic78 at gmail dot com>  * Copyright (C) 2009-2010 Dawit Alemayehu <adawit at kde dot org>  * @@ -45,7 +45,6 @@  #include "mainview.h"  #include "mainwindow.h"  #include "networkaccessmanager.h" -#include "newtabpage.h"  #include "urlbar.h"  #include "webpluginfactory.h"  #include "websnap.h" @@ -615,16 +614,6 @@ void WebPage::showSSLInfo(QPoint pos)  } -void WebPage::updateImage(bool ok) -{ -    if (ok) -    { -        NewTabPage p(mainFrame()); -        p.updateThumbs(); -    } -} - -  void WebPage::copyToTempFileResult(KJob* job)  {      if (job->error()) diff --git a/src/webpage.h b/src/webpage.h index 6a717f78..7e5df309 100644 --- a/src/webpage.h +++ b/src/webpage.h @@ -6,7 +6,7 @@  * Copyright (C) 2008 Dirk Mueller <mueller@kde.org>  * Copyright (C) 2008 Urs Wolfer <uwolfer @ kde.org>  * Copyright (C) 2008 Michael Howell <mhowell123@gmail.com> -* Copyright (C) 2008-2011 by Andrea Diamantini <adjam7 at gmail dot com> +* Copyright (C) 2008-2012 by Andrea Diamantini <adjam7 at gmail dot com>  * Copyright (C) 2010 by Matthieu Gicquel <matgic78 at gmail dot com>  *  * @@ -103,7 +103,6 @@ private Q_SLOTS:      void loadStarted();      void loadFinished(bool);      void showSSLInfo(QPoint); -    void updateImage(bool ok);      void copyToTempFileResult(KJob*); | 
