From 682cac6c2ce4cf21b9a2f4314cc452060f389b20 Mon Sep 17 00:00:00 2001 From: Andrea Diamantini Date: Mon, 14 Dec 2009 00:00:19 +0100 Subject: Just some stupid fixes --- src/bookmarks/bookmarkspanel.cpp | 52 +++-- src/bookmarks/bookmarkspanel.h | 5 +- src/bookmarks/bookmarksproxy.cpp | 59 +++-- src/bookmarks/bookmarkstreemodel.cpp | 430 ++++++++++++++++++++--------------- 4 files changed, 323 insertions(+), 223 deletions(-) (limited to 'src/bookmarks') diff --git a/src/bookmarks/bookmarkspanel.cpp b/src/bookmarks/bookmarkspanel.cpp index a09e0058..97097fbb 100644 --- a/src/bookmarks/bookmarkspanel.cpp +++ b/src/bookmarks/bookmarkspanel.cpp @@ -24,8 +24,11 @@ * ============================================================ */ -// rekonq includes +// Self Includes #include "bookmarkspanel.h" +#include "bookmarkspanel.moc" + +// Local Includes #include "bookmarkstreemodel.h" #include "bookmarksproxy.h" @@ -42,11 +45,11 @@ #include #include -BookmarksPanel::BookmarksPanel(const QString &title, QWidget *parent, Qt::WindowFlags flags): - QDockWidget(title, parent, flags) -{ - setup(); +BookmarksPanel::BookmarksPanel(const QString &title, QWidget *parent, Qt::WindowFlags flags) + : QDockWidget(title, parent, flags) +{ + setup(); setShown(ReKonfig::showBookmarksPanel()); } @@ -54,22 +57,23 @@ BookmarksPanel::BookmarksPanel(const QString &title, QWidget *parent, Qt::Window BookmarksPanel::~BookmarksPanel() { ReKonfig::setShowBookmarksPanel(!isHidden()); - - delete ui; + delete ui; } + void BookmarksPanel::bookmarkActivated( const QModelIndex &index ) { - if( index.isValid() ) - emit openUrl( qVariantValue< KUrl >( index.data( Qt::UserRole ) ) ); + if( index.isValid() ) + emit openUrl( qVariantValue< KUrl >( index.data( Qt::UserRole ) ) ); } + void BookmarksPanel::setup() { - setObjectName("bookmarksPanel"); + setObjectName("bookmarksPanel"); setAllowedAreas(Qt::LeftDockWidgetArea | Qt::RightDockWidgetArea); - ui = new QWidget(this); + ui = new QWidget(this); // setup search bar QHBoxLayout *searchLayout = new QHBoxLayout; @@ -79,16 +83,16 @@ void BookmarksPanel::setup() KLineEdit *search = new KLineEdit; search->setClearButtonShown(true); searchLayout->addWidget(search); - searchLabel->setBuddy( search ); + searchLabel->setBuddy( search ); - // setup tree view - QTreeView *treeView = new QTreeView(ui); + // setup tree view + QTreeView *treeView = new QTreeView(ui); treeView->setUniformRowHeights(true); treeView->setSelectionBehavior(QAbstractItemView::SelectRows); treeView->setTextElideMode(Qt::ElideMiddle); treeView->setAlternatingRowColors(true); - treeView->header()->hide(); - treeView->setRootIsDecorated( false ); + treeView->header()->hide(); + treeView->setRootIsDecorated( false ); // put everything together QVBoxLayout *vBoxLayout = new QVBoxLayout; @@ -96,15 +100,15 @@ void BookmarksPanel::setup() vBoxLayout->addLayout(searchLayout); vBoxLayout->addWidget(treeView); - // add it to the UI + // add it to the UI ui->setLayout(vBoxLayout); - setWidget(ui); + setWidget(ui); - BookmarksTreeModel *model = new BookmarksTreeModel( this ); - BookmarksProxy *proxy = new BookmarksProxy(ui); - proxy->setSourceModel( model ); - treeView->setModel( proxy ); + BookmarksTreeModel *model = new BookmarksTreeModel( this ); + BookmarksProxy *proxy = new BookmarksProxy(ui); + proxy->setSourceModel( model ); + treeView->setModel( proxy ); - connect(search, SIGNAL(textChanged(QString)), proxy, SLOT(setFilterFixedString(QString))); - connect( treeView, SIGNAL( activated(QModelIndex) ), this, SLOT( bookmarkActivated(QModelIndex) ) ); + connect(search, SIGNAL(textChanged(QString)), proxy, SLOT(setFilterFixedString(QString))); + connect( treeView, SIGNAL( activated(QModelIndex) ), this, SLOT( bookmarkActivated(QModelIndex) ) ); } diff --git a/src/bookmarks/bookmarkspanel.h b/src/bookmarks/bookmarkspanel.h index 8c3e6121..b7c0b5ed 100644 --- a/src/bookmarks/bookmarkspanel.h +++ b/src/bookmarks/bookmarkspanel.h @@ -34,6 +34,7 @@ class KUrl; class QModelIndex; + class BookmarksPanel : public QDockWidget { Q_OBJECT @@ -47,10 +48,10 @@ signals: void openUrl(const KUrl &); private slots: - void bookmarkActivated( const QModelIndex &index ); + void bookmarkActivated( const QModelIndex &index ); private: - void setup(); + void setup(); QWidget *ui; }; diff --git a/src/bookmarks/bookmarksproxy.cpp b/src/bookmarks/bookmarksproxy.cpp index 87d1ce71..e94fbeff 100644 --- a/src/bookmarks/bookmarksproxy.cpp +++ b/src/bookmarks/bookmarksproxy.cpp @@ -1,29 +1,56 @@ +/* ============================================================ +* +* This file is a part of the rekonq project +* +* Copyright (C) 2009 by Nils Weigel +* +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License or (at your option) version 3 or any later version +* accepted by the membership of KDE e.V. (or its successor approved +* by the membership of KDE e.V.), which shall act as a proxy +* defined in Section 14 of version 3 of the license. +* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program. If not, see . +* +* ============================================================ */ + + +// Self Includes #include "bookmarksproxy.h" +#include "bookmarksproxy.moc" -BookmarksProxy::BookmarksProxy( QObject *parent ): - QSortFilterProxyModel( parent ) + +BookmarksProxy::BookmarksProxy( QObject *parent ) + : QSortFilterProxyModel( parent ) { } + bool BookmarksProxy::filterAcceptsRow( int source_row, const QModelIndex &source_parent ) const { - QModelIndex idx = sourceModel()->index( source_row, 0, source_parent ); - -// return idx.data().toString().contains( filterRegExp() ); - return recursiveMatch( idx ); + QModelIndex idx = sourceModel()->index( source_row, 0, source_parent ); + return recursiveMatch( idx ); } + bool BookmarksProxy::recursiveMatch( const QModelIndex &index ) const { - if( index.data().toString().contains( filterRegExp() ) ) { - return true; - } - - for( int childRow = 0; childRow < sourceModel()->rowCount( index ); ++childRow ) { - if( recursiveMatch( sourceModel()->index( childRow, 0, index ) ) ) { - return true; - } - } + if( index.data().toString().contains( filterRegExp() ) ) + return true; - return false; + for( int childRow = 0; childRow < sourceModel()->rowCount( index ); ++childRow ) + { + if( recursiveMatch( sourceModel()->index( childRow, 0, index ) ) ) + return true; + } + return false; } diff --git a/src/bookmarks/bookmarkstreemodel.cpp b/src/bookmarks/bookmarkstreemodel.cpp index b5690a9b..140fa9c8 100644 --- a/src/bookmarks/bookmarkstreemodel.cpp +++ b/src/bookmarks/bookmarkstreemodel.cpp @@ -1,6 +1,34 @@ +/* ============================================================ +* +* This file is a part of the rekonq project +* +* Copyright (C) 2009 by Nils Weigel +* +* +* This program is free software; you can redistribute it and/or +* modify it under the terms of the GNU General Public License as +* published by the Free Software Foundation; either version 2 of +* the License or (at your option) version 3 or any later version +* accepted by the membership of KDE e.V. (or its successor approved +* by the membership of KDE e.V.), which shall act as a proxy +* defined in Section 14 of version 3 of the license. +* +* This program is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU General Public License for more details. +* +* You should have received a copy of the GNU General Public License +* along with this program. If not, see . +* +* ============================================================ */ + + +// Self Includes #include "bookmarkstreemodel.h" +#include "bookmarkstreemodel.moc" -// rekonq includes +// Local Includes #include "application.h" #include "bookmarksmanager.h" @@ -8,248 +36,288 @@ #include #include + class BookmarksTreeModel::BtmItem { public: - BtmItem(const KBookmark &bm): - m_parent(0), m_kbm(bm) - { - } - ~BtmItem() - { - qDeleteAll(m_children); - } - - QVariant data( int role = Qt::DisplayRole ) const - { - if( m_kbm.isNull() ) - return QVariant();// should only happen for root item - - if( role == Qt::DisplayRole ) - return m_kbm.text(); - if( role == Qt::DecorationRole ) - return KIcon( m_kbm.icon() ); - if( role == Qt::UserRole ) - return m_kbm.url(); - - return QVariant(); - } - - int row() const - { - if(m_parent) - return m_parent->m_children.indexOf( const_cast< BtmItem* >( this ) ); - return 0; - } - int childCount() const - { - return m_children.count(); - } - BtmItem* child( int n ) - { - Q_ASSERT(n>=0); - Q_ASSERT(nm_parent = this; - m_children << child; - } - void clear() - { - qDeleteAll(m_children); - m_children.clear(); - } + BtmItem(const KBookmark &bm) + : m_parent(0) + , m_kbm(bm) + { + } + + + ~BtmItem() + { + qDeleteAll(m_children); + } + + + QVariant data( int role = Qt::DisplayRole ) const + { + if( m_kbm.isNull() ) + return QVariant(); // should only happen for root item + + if( role == Qt::DisplayRole ) + return m_kbm.text(); + if( role == Qt::DecorationRole ) + return KIcon( m_kbm.icon() ); + if( role == Qt::UserRole ) + return m_kbm.url(); + + return QVariant(); + } + + + int row() const + { + if(m_parent) + return m_parent->m_children.indexOf( const_cast< BtmItem* >( this ) ); + return 0; + } + + + int childCount() const + { + return m_children.count(); + } -private: - BtmItem *m_parent; - QList< BtmItem* > m_children; - KBookmark m_kbm; + BtmItem* child( int n ) + { + Q_ASSERT(n>=0); + Q_ASSERT(nm_parent = this; + m_children << child; + } + + + void clear() + { + qDeleteAll(m_children); + m_children.clear(); + } + +private: + BtmItem *m_parent; + QList< BtmItem* > m_children; + KBookmark m_kbm; }; -BookmarksTreeModel::BookmarksTreeModel(QObject *parent): - QAbstractItemModel(parent), m_root(0) + +// ------------------------------------------------------------------------------------- + + +BookmarksTreeModel::BookmarksTreeModel(QObject *parent) + : QAbstractItemModel(parent) + , m_root(0) { - resetModel(); - connect( Application::bookmarkProvider()->bookmarkManager(), SIGNAL( changed(QString,QString) ), this, SLOT( bookmarksChanged(QString) ) ); - connect( Application::bookmarkProvider()->bookmarkManager(), SIGNAL( bookmarksChanged(QString) ), this, SLOT( bookmarksChanged(QString) ) ); + resetModel(); + connect( Application::bookmarkProvider()->bookmarkManager(), SIGNAL( changed(QString,QString) ), this, SLOT( bookmarksChanged(QString) ) ); + connect( Application::bookmarkProvider()->bookmarkManager(), SIGNAL( bookmarksChanged(QString) ), this, SLOT( bookmarksChanged(QString) ) ); } + BookmarksTreeModel::~BookmarksTreeModel() { - delete m_root; + delete m_root; } + int BookmarksTreeModel::rowCount(const QModelIndex &parent) const { - BtmItem *parentItem = 0; - if( !parent.isValid() ) { - parentItem = m_root; - } - else { - parentItem = static_cast< BtmItem* >( parent.internalPointer() ); - } - - return parentItem->childCount(); + BtmItem *parentItem = 0; + if( !parent.isValid() ) + { + parentItem = m_root; + } + else + { + parentItem = static_cast< BtmItem* >( parent.internalPointer() ); + } + + return parentItem->childCount(); } -int BookmarksTreeModel::columnCount(const QModelIndex &/*parent*/) const + +int BookmarksTreeModel::columnCount(const QModelIndex &parent) const { - // name - return 1; + Q_UNUSED(parent) + // name + return 1; } + QVariant BookmarksTreeModel::headerData(int section, Qt::Orientation orientation, int role) const { - if( orientation == Qt::Horizontal && role == Qt::DisplayRole && section == 0 ) - return i18n( "Bookmark" ); + if( orientation == Qt::Horizontal + && role == Qt::DisplayRole + && section == 0 + ) + return i18n( "Bookmark" ); - return QVariant(); + return QVariant(); } -Qt::ItemFlags BookmarksTreeModel::flags(const QModelIndex &/*index*/) const + +Qt::ItemFlags BookmarksTreeModel::flags(const QModelIndex &index) const { - return Qt::ItemIsEnabled|Qt::ItemIsSelectable; + Q_UNUSED(index) + return Qt::ItemIsEnabled | Qt::ItemIsSelectable; } + QModelIndex BookmarksTreeModel::index(int row, int column, const QModelIndex &parent) const { - if( !hasIndex( row, column, parent ) ) { - return QModelIndex(); - } - - BtmItem *parentItem; - - if( !parent.isValid() ) { - parentItem = m_root; - } - else { - parentItem = static_cast< BtmItem* >( parent.internalPointer() ); - } - - BtmItem *childItem = parentItem->child( row ); - if( childItem ) { - return createIndex( row, column, childItem ); - } - - return QModelIndex(); + if( !hasIndex( row, column, parent ) ) + { + return QModelIndex(); + } + + BtmItem *parentItem; + + if( !parent.isValid() ) + { + parentItem = m_root; + } + else + { + parentItem = static_cast< BtmItem* >( parent.internalPointer() ); + } + + BtmItem *childItem = parentItem->child( row ); + if( childItem ) + { + return createIndex( row, column, childItem ); + } + + return QModelIndex(); } + QModelIndex BookmarksTreeModel::parent(const QModelIndex &index) const { - if( !index.isValid() ) { - return QModelIndex(); - } + if( !index.isValid() ) + { + return QModelIndex(); + } - BtmItem *childItem = static_cast< BtmItem* >( index.internalPointer() ); - BtmItem *parentItem = childItem->parent(); + BtmItem *childItem = static_cast< BtmItem* >( index.internalPointer() ); + BtmItem *parentItem = childItem->parent(); - if( parentItem == m_root ) { - return QModelIndex(); - } + if( parentItem == m_root ) + { + return QModelIndex(); + } - return createIndex( parentItem->row(), 0, parentItem ); + return createIndex( parentItem->row(), 0, parentItem ); } + QVariant BookmarksTreeModel::data(const QModelIndex &index, int role) const { - if( !index.isValid() ) { - return QVariant(); - } - - BtmItem *node = static_cast< BtmItem* >( index.internalPointer() ); - if( node && node == m_root ) { - if( role == Qt::DisplayRole ) - return i18n( "Bookmarks" ); - else if( role == Qt::DecorationRole ) - return KIcon( "bookmarks" ); - } - else if( node ) { - return node->data( role ); - } - - return QVariant(); + if( !index.isValid() ) + { + return QVariant(); + } + + BtmItem *node = static_cast< BtmItem* >( index.internalPointer() ); + if( node && node == m_root ) + { + if( role == Qt::DisplayRole ) + return i18n( "Bookmarks" ); + if( role == Qt::DecorationRole ) + return KIcon( "bookmarks" ); + } + else + { + if( node ) + return node->data( role ); + } + + return QVariant(); } -// bool BookmarksTreeModel::setData(const QModelIndex &index, const QVariant &value, int role) -// { -// } void BookmarksTreeModel::bookmarksChanged( const QString &groupAddress ) { -// qDebug( "bookmarksChanged '%s'", qPrintable( groupAddress ) ); - - if( groupAddress.isEmpty() ) { - resetModel(); - return; - } - - BtmItem *node = m_root; - QModelIndex nodeIndex; - - QStringList indexChain( groupAddress.split( '/', QString::SkipEmptyParts) ); - foreach( QString sIndex, indexChain ) { - bool ok; - int i = sIndex.toInt( &ok ); - if( !ok ) - break; - - if( i < 0 || i >= node->childCount() ) - break; - - node = node->child( i ); - nodeIndex = index( i, 0, nodeIndex ); - } -// qDebug( " changed: '%s'(0-%d)", ( node == m_root ? "ROOT" : qPrintable( node->data().toString() ) ), node->childCount() ); - emit dataChanged( index( 0, 0, nodeIndex ), index( node->childCount(), 0, nodeIndex ) ); + if( groupAddress.isEmpty() ) + { + resetModel(); + return; + } + + BtmItem *node = m_root; + QModelIndex nodeIndex; + + QStringList indexChain( groupAddress.split( '/', QString::SkipEmptyParts) ); + foreach( QString sIndex, indexChain ) + { + bool ok; + int i = sIndex.toInt( &ok ); + if( !ok ) + break; + + if( i < 0 || i >= node->childCount() ) + break; + + node = node->child( i ); + nodeIndex = index( i, 0, nodeIndex ); + } + emit dataChanged( index( 0, 0, nodeIndex ), index( node->childCount(), 0, nodeIndex ) ); } + void BookmarksTreeModel::resetModel() { - setRoot(Application::bookmarkProvider()->rootGroup()); + setRoot(Application::bookmarkProvider()->rootGroup()); } + void BookmarksTreeModel::setRoot(KBookmarkGroup bmg) { - delete m_root; - m_root = new BtmItem(KBookmark()); + delete m_root; + m_root = new BtmItem(KBookmark()); - if( bmg.isNull() ) { - return; - } + if( bmg.isNull() ) + return; - populate( m_root, bmg ); - - reset(); + populate( m_root, bmg ); + reset(); } + void BookmarksTreeModel::populate( BtmItem *node, KBookmarkGroup bmg) { - node->clear(); + node->clear(); - if( bmg.isNull() ) { - return; - } + if( bmg.isNull() ) + return; - KBookmark bm = bmg.first(); - while( !bm.isNull() ) { - BtmItem *newChild = new BtmItem( bm ); - if( bm.isGroup() ) - populate( newChild, bm.toGroup() ); + KBookmark bm = bmg.first(); + while( !bm.isNull() ) + { + BtmItem *newChild = new BtmItem( bm ); + if( bm.isGroup() ) + populate( newChild, bm.toGroup() ); - node->appendChild( newChild ); - bm = bmg.next( bm ); - } + node->appendChild( newChild ); + bm = bmg.next( bm ); + } } -- cgit v1.2.1