diff options
author | aqua <aqua@iserlohn-fortress.net> | 2022-09-10 16:47:56 +0300 |
---|---|---|
committer | aqua <aqua@iserlohn-fortress.net> | 2022-09-14 08:51:10 +0300 |
commit | 3d8255c70e71e3d4d23a7c798c94b16ab7e970f6 (patch) | |
tree | f191d578d39640a7c01b8c0df974c453272ede55 | |
parent | Migrate BookmarkManager (diff) | |
download | rekonq-3d8255c70e71e3d4d23a7c798c94b16ab7e970f6.tar.xz |
Add Bookmarks panel
-rw-r--r-- | CMakeLists.txt | 2 | ||||
-rw-r--r-- | src/CMakeLists.txt | 3 | ||||
-rw-r--r-- | src/application.cpp | 54 | ||||
-rw-r--r-- | src/application.hpp | 2 | ||||
-rw-r--r-- | src/panels/bookmarkspanel.cpp | 179 | ||||
-rw-r--r-- | src/panels/bookmarkspanel.h | 93 | ||||
-rw-r--r-- | src/panels/bookmarkspanel.hpp | 27 | ||||
-rw-r--r-- | src/rekonq.kcfg | 2 | ||||
-rw-r--r-- | src/rekonqwindow.cpp | 25 | ||||
-rw-r--r-- | src/rekonqwindow.hpp | 5 | ||||
-rw-r--r-- | src/rekonqwindow.ui | 53 | ||||
-rw-r--r-- | src/rekonqwindow_class.cpp | 9 | ||||
-rw-r--r-- | src/test/application_mock.hpp | 1 |
13 files changed, 143 insertions, 312 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index 0ee5e07e..69876ac4 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -116,7 +116,7 @@ add_executable(rekonq ${rekonq_SRCS} third-party/resources.qrc) target_include_directories(rekonq PRIVATE src) target_link_libraries(rekonq spdlog::spdlog Qt6::Widgets SingleApplication::SingleApplication - settings pluginloader + bookmarks settings pluginloader ) add_custom_target(rekonq_check_license python scripts/check_license.py ${rekonq_SRCS} WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}) diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 77155bba..e3a03dba 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -11,6 +11,7 @@ set(rekonqwindow_SRCS rekonqwindow_class.cpp rekonqwindow.hpp rekonqwindow.ui tabbar/tabbar.cpp tabbar/tabbar.h urlbar/urlbar.cpp urlbar/urlbar.hpp urlbar/completer.cpp urlbar/completer.hpp + panels/bookmarkspanel.cpp panels/bookmarkspanel.hpp ) set(rekonqwindow_IFACES ${PROJECT_SOURCE_DIR}/include/rview.hpp) @@ -126,7 +127,7 @@ if(${CMAKE_BUILD_TYPE} STREQUAL "Debug") add_executable(test_rekonqwindow test/test_rekonqwindow.cpp test/rekonqwindow_mock.hpp test/settings_mock.hpp ${rekonqwindow_SRCS} ${rekonqwindow_IFACES}) target_compile_definitions(test_rekonqwindow PRIVATE REKONQ_TEST) - target_link_libraries(test_rekonqwindow GTest::gtest GTest::gmock settings) + target_link_libraries(test_rekonqwindow GTest::gtest GTest::gmock bookmarks settings) gtest_discover_tests(test_rekonqwindow) endif() diff --git a/src/application.cpp b/src/application.cpp index 38ad7477..0f86feec 100644 --- a/src/application.cpp +++ b/src/application.cpp @@ -5,13 +5,11 @@ * Copyright (C) 2012-2013 by Andrea Diamantini <adjam7 at gmail dot com> * SPDX-License-Identifier: GPL-3.0-only * Copyright (C) 2022 aqua <aqua@iserlohn-fortress.net> - * ============================================================ - * Description: Application Main Class * ============================================================ */ #include "application.hpp" +#include "bookmarks/bookmarkstreemodel.hpp" #include "plugins/pluginloader.h" -#include "plugins/rplugininterface.hpp" #include "rekonqwindow.hpp" #include "settings/settings.hpp" #include "settings/settingsdialog.h" @@ -59,11 +57,6 @@ Application::Application(int &argc, char *argv[]) : SingleApplication(argc, argv Application::~Application() { - // ok, we are closing well: don't recover on next load.. - // ReKonfig::setRecoverOnCrash(0); - // saveConfiguration(); - delete m_settings; - // Destroy all windows... for (const auto &window : m_windows) delete window; @@ -75,6 +68,11 @@ Application::~Application() plugin->unload(); delete plugin; } + + // ok, we are closing well: don't recover on next load.. + // ReKonfig::setRecoverOnCrash(0); + // saveConfiguration(); + delete m_settings; } // --------------------------------------------------------------------------------------------------------------- @@ -95,7 +93,31 @@ bool Application::registerPlugin(const QString &path) m_plugins.append(loader); return true; } + // --------------------------------------------------------------------------------------------------------------- +// Bookmarks + +struct BookmarksModelPrivate { + + BookmarksModelPrivate() + { + model = new BookmarkModel; + const auto path = QStandardPaths::writableLocation(QStandardPaths::AppLocalDataLocation) + "/bookmarks.xbel"; + spdlog::debug("Loading bookmarks from {}", qUtf8Printable(path)); + QFile f(path); + if (f.open(QIODevice::ReadOnly | QIODevice::Text)) { + model->load(&f); + f.close(); + } + } + ~BookmarksModelPrivate() noexcept { delete model; } + + BookmarkModel *model; +}; +Q_GLOBAL_STATIC(BookmarksModelPrivate, bookmarks_d); + +QAbstractItemModel *Application::bookmarks() const { return bookmarks_d->model; } + /* int Application::newInstance() { @@ -409,22 +431,8 @@ RekonqView *Application::newView(const QUrl &url, RekonqWindow *window) return view; } -/* -RekonqWindowList Application::rekonqWindowList() -{ - return m_rekonqWindows; -} - - -WebAppList Application::webAppList() -{ - return m_webApps; -} - - // ------------------------------------------------------------------------------------------------------------------ - - +/* bool Application::eventFilter(QObject* watched, QEvent* event) { // Track which window was activated most recently to prefer it on window choosing diff --git a/src/application.hpp b/src/application.hpp index f435ef8d..c7401563 100644 --- a/src/application.hpp +++ b/src/application.hpp @@ -22,6 +22,7 @@ class RekonqView; class RekonqWindow; class PluginLoader; class RekonqSettings; +class QAbstractItemModel; using RekonqPluginList = QList<QPointer<PluginLoader>>; using RekonqWindowList = QList<QPointer<RekonqWindow>>; @@ -45,6 +46,7 @@ public: // int newInstance(); static Application *instance() { return (qobject_cast<Application *>(QCoreApplication::instance())); } [[nodiscard]] auto *settings() const { return m_settings; } + REKONQ_TEST_VIRTUAL QAbstractItemModel *bookmarks() const REKONQ_TEST_PURE; // RekonqWindow *rekonqWindow(const QString &activityID = QString()); [[nodiscard]] auto windowList() const { return m_windows; } diff --git a/src/panels/bookmarkspanel.cpp b/src/panels/bookmarkspanel.cpp index 57796a23..ee7ae717 100644 --- a/src/panels/bookmarkspanel.cpp +++ b/src/panels/bookmarkspanel.cpp @@ -1,170 +1,25 @@ /* ============================================================ -* -* This file is a part of the rekonq project -* -* Copyright (C) 2009 by Nils Weigel <nehlsen at gmail dot com> -* Copyright (C) 2010-2013 by Andrea Diamantini <adjam7 at gmail dot com> -* Copyright (C) 2010 by Yoann Laissus <yoann dot laissus at gmail dot com> -* -* 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 <http://www.gnu.org/licenses/>. -* -* ============================================================ */ + * The rekonq project + * ============================================================ + * SPDX-License-Identifier: GPL-3.0-only + * Copyright (C) 2022 aqua <aqua@iserlohn-fortress.net> + * ============================================================ */ +#include "bookmarkspanel.hpp" +#include "bookmarks/bookmarkstreemodel.hpp" -// Self Includes -#include "bookmarkspanel.h" -#include "bookmarkspanel.moc" - -// Auto Includes -#include "rekonq.h" - -// Local Includes -#include "bookmarkmanager.h" -#include "bookmarkstreemodel.h" -#include "bookmarkscontextmenu.h" -#include "bookmarkowner.h" - -#include "paneltreeview.h" -#include "urlfilterproxymodel.h" - - -BookmarksPanel::BookmarksPanel(const QString &title, QWidget *parent, Qt::WindowFlags flags) - : UrlPanel(title, parent, flags) - , _bkTreeModel(new BookmarksTreeModel(this)) - , _loadingState(false) -{ - setObjectName("bookmarksPanel"); - setVisible(ReKonfig::showBookmarksPanel()); - - panelTreeView()->setDragEnabled(true); - panelTreeView()->setAcceptDrops(true); - connect(_bkTreeModel, SIGNAL(bookmarksUpdated()), this, SLOT(loadFoldedState())); -} - - -BookmarksPanel::~BookmarksPanel() -{ - ReKonfig::setShowBookmarksPanel(!isHidden()); -} - - -void BookmarksPanel::loadFoldedState() -{ - _loadingState = true; - loadFoldedState(QModelIndex()); - _loadingState = false; -} - - -void BookmarksPanel::contextMenu(const QPoint &pos) -{ - if (_loadingState) - return; - - BookmarksContextMenu menu(bookmarkForIndex(panelTreeView()->indexAt(pos)), - BookmarkManager::self()->manager(), - BookmarkManager::self()->owner() - ); - - menu.exec(panelTreeView()->mapToGlobal(pos)); -} - - -void BookmarksPanel::deleteBookmark() -{ - QModelIndex index = panelTreeView()->currentIndex(); - if (_loadingState || !index.isValid()) - return; - - BookmarkManager::self()->owner()->deleteBookmark(bookmarkForIndex(index)); -} - - -void BookmarksPanel::onCollapse(const QModelIndex &index) -{ - if (_loadingState) - return; - - bookmarkForIndex(index).internalElement().setAttribute("folded", "yes"); - emit expansionChanged(); -} - - -void BookmarksPanel::onExpand(const QModelIndex &index) +BookmarksPanel::BookmarksPanel(QWidget *parent) : QTreeView(parent) { - if (_loadingState) - return; - - bookmarkForIndex(index).internalElement().setAttribute("folded", "no"); - emit expansionChanged(); -} - - -void BookmarksPanel::setup() -{ - UrlPanel::setup(); - - connect(panelTreeView(), SIGNAL(delKeyPressed()), this, SLOT(deleteBookmark())); - connect(panelTreeView(), SIGNAL(collapsed(QModelIndex)), this, SLOT(onCollapse(QModelIndex))); - connect(panelTreeView(), SIGNAL(expanded(QModelIndex)), this, SLOT(onExpand(QModelIndex))); - - loadFoldedState(); + connect(this, &QTreeView::activated, this, [this](const QModelIndex &index) { + auto *item = model()->item(index); + const auto url = item->data(BookmarksTreeItem::Href).toUrl(); + emit loadUrl(url, rekonq::CurrentTab); + }); } - -void BookmarksPanel::loadFoldedState(const QModelIndex &root) -{ - QAbstractItemModel *model = panelTreeView()->model(); - if (!model) - return; - - int count = model->rowCount(root); - QModelIndex index; - - for (int i = 0; i < count; ++i) - { - index = model->index(i, 0, root); - if (index.isValid()) - { - KBookmark bm = bookmarkForIndex(index); - if (bm.isGroup()) - { - panelTreeView()->setExpanded(index, bm.toGroup().isOpen()); - loadFoldedState(index); - } - } - } -} - - -KBookmark BookmarksPanel::bookmarkForIndex(const QModelIndex &index) -{ - if (!index.isValid()) - return KBookmark(); - - const UrlFilterProxyModel *proxyModel = static_cast<const UrlFilterProxyModel*>(index.model()); - QModelIndex originalIndex = proxyModel->mapToSource(index); - - BtmItem *node = static_cast<BtmItem*>(originalIndex.internalPointer()); - return node->getBkm(); -} - - -QAbstractItemModel* BookmarksPanel::model() +BookmarkModel *BookmarksPanel::model() const { - return _bkTreeModel; + auto *m = qobject_cast<BookmarkModel *>(QTreeView::model()); + Q_CHECK_PTR(m); + return m; } diff --git a/src/panels/bookmarkspanel.h b/src/panels/bookmarkspanel.h deleted file mode 100644 index 8c827c65..00000000 --- a/src/panels/bookmarkspanel.h +++ /dev/null @@ -1,93 +0,0 @@ -/* ============================================================ -* -* This file is a part of the rekonq project -* -* Copyright (C) 2009 by Nils Weigel <nehlsen at gmail dot com> -* Copyright (C) 2010-2013 by Andrea Diamantini <adjam7 at gmail dot com> -* Copyright (C) 2010 by Yoann Laissus <yoann dot laissus at gmail dot com> -* -* -* 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 <http://www.gnu.org/licenses/>. -* -* ============================================================ */ - - -#ifndef BOOKMARKS_PANEL_H -#define BOOKMARKS_PANEL_H - - -// Rekonq Includes -#include "rekonq_defines.h" - -// Local Includes -#include "urlpanel.h" - -// Forward Declarations -class BookmarksTreeModel; - -class KBookmark; -class QModelIndex; - - -class REKONQ_TESTS_EXPORT BookmarksPanel : public UrlPanel -{ - Q_OBJECT - -public: - explicit BookmarksPanel(const QString &title, QWidget *parent = 0, Qt::WindowFlags flags = 0); - ~BookmarksPanel(); - -public Q_SLOTS: - void loadFoldedState(); - -Q_SIGNALS: - void expansionChanged(); - -private Q_SLOTS: - void contextMenu(const QPoint &pos); - - virtual void contextMenuItem(const QPoint &pos) - { - contextMenu(pos); - } - virtual void contextMenuGroup(const QPoint &pos) - { - contextMenu(pos); - } - virtual void contextMenuEmpty(const QPoint &pos) - { - contextMenu(pos); - } - - void deleteBookmark(); - void onCollapse(const QModelIndex &index); - void onExpand(const QModelIndex &index); - -private: - virtual void setup(); - - void loadFoldedState(const QModelIndex &root); - - KBookmark bookmarkForIndex(const QModelIndex &index); - - virtual QAbstractItemModel* model(); - - BookmarksTreeModel *_bkTreeModel; - bool _loadingState; -}; - -#endif // BOOKMARKS_PANEL_H diff --git a/src/panels/bookmarkspanel.hpp b/src/panels/bookmarkspanel.hpp new file mode 100644 index 00000000..5961af7d --- /dev/null +++ b/src/panels/bookmarkspanel.hpp @@ -0,0 +1,27 @@ +/* ============================================================ + * The rekonq project + * ============================================================ + * SPDX-License-Identifier: GPL-3.0-only + * Copyright (C) 2022 aqua <aqua@iserlohn-fortress.net> + * ============================================================ */ + +#pragma once + +#include "rekonq.hpp" +#include <QTreeView> + +class BookmarkModel; + +class BookmarksPanel : public QTreeView { + Q_OBJECT + +public: + explicit BookmarksPanel(QWidget *parent = nullptr); + ~BookmarksPanel() = default; + +signals: + void loadUrl(const QUrl &url, rekonq::OpenType type); + +private: + [[nodiscard]] BookmarkModel *model() const; +}; diff --git a/src/rekonq.kcfg b/src/rekonq.kcfg index 7776a0b1..52a5d657 100644 --- a/src/rekonq.kcfg +++ b/src/rekonq.kcfg @@ -95,7 +95,7 @@ <entry name="Task Manager" key="actionTaskManager" type="Shortcut"> <default></default> </entry> - <entry name="Show sidebar" key="actionShowSidebar" type="Shortcut"> + <entry name="Show Bookmarks Panel" key="actionShowBookmarksPanel" type="Shortcut"> <default>Ctrl+B</default> </entry> <entry name="Show search bar" key="actionShowSearchBar" type="Shortcut"> diff --git a/src/rekonqwindow.cpp b/src/rekonqwindow.cpp index f9f26186..461de12e 100644 --- a/src/rekonqwindow.cpp +++ b/src/rekonqwindow.cpp @@ -90,31 +90,6 @@ void RekonqWindow::loadUrl(const QUrl &url, rekonq::OpenType type) // -------------------------------------------------------------------------------------------------- /* -void RekonqWindow::showBookmarksPanel(bool on) -{ - if (on) - { - if (_bookmarksPanel.isNull()) - { - _bookmarksPanel = new BookmarksPanel(i18n("Bookmarks Panel"), this); - connect(_bookmarksPanel.data(), SIGNAL(openUrl(KUrl,Rekonq::OpenType)), this, -SLOT(loadUrl(KUrl,Rekonq::OpenType))); - - QAction *a = _tabWidget->actionByName(QL1S("show_bookmarks_panel")); - connect(_bookmarksPanel.data(), SIGNAL(visibilityChanged(bool)), a, SLOT(setChecked(bool))); - } - _splitter->insertWidget(0, _bookmarksPanel.data()); - _bookmarksPanel.data()->show(); - } - else - { - _bookmarksPanel.data()->hide(); - delete _bookmarksPanel.data(); - _bookmarksPanel.clear(); - } -} - - void RekonqWindow::showHistoryPanel(bool on) { if (on) diff --git a/src/rekonqwindow.hpp b/src/rekonqwindow.hpp index 849bdff9..04287a90 100644 --- a/src/rekonqwindow.hpp +++ b/src/rekonqwindow.hpp @@ -39,12 +39,9 @@ public slots: // void loadUrl(const KUrl &, Rekonq::OpenType type = Rekonq::CurrentTab, TabHistory *history = 0); private slots: - // void showBookmarksPanel(bool); + void showBookmarksPanel(bool); // void showHistoryPanel(bool); private: Ui::RekonqWindow *ui; - - // QWeakPointer<HistoryPanel> _historyPanel; - // QWeakPointer<BookmarksPanel> _bookmarksPanel; }; diff --git a/src/rekonqwindow.ui b/src/rekonqwindow.ui index b16fe187..64e1a517 100644 --- a/src/rekonqwindow.ui +++ b/src/rekonqwindow.ui @@ -110,7 +110,39 @@ </layout> </item> <item> - <widget class="QStackedWidget" name="views"/> + <widget class="QSplitter" name="splitter"> + <property name="sizePolicy"> + <sizepolicy hsizetype="Expanding" vsizetype="Expanding"> + <horstretch>0</horstretch> + <verstretch>0</verstretch> + </sizepolicy> + </property> + <property name="orientation"> + <enum>Qt::Horizontal</enum> + </property> + <widget class="BookmarksPanel" name="bookmarksPanel"> + <property name="sizePolicy"> + <sizepolicy hsizetype="Preferred" vsizetype="Expanding"> + <horstretch>0</horstretch> + <verstretch>0</verstretch> + </sizepolicy> + </property> + <property name="maximumSize"> + <size> + <width>400</width> + <height>16777215</height> + </size> + </property> + </widget> + <widget class="QStackedWidget" name="views"> + <property name="sizePolicy"> + <sizepolicy hsizetype="Expanding" vsizetype="Expanding"> + <horstretch>0</horstretch> + <verstretch>0</verstretch> + </sizepolicy> + </property> + </widget> + </widget> </item> </layout> </widget> @@ -120,7 +152,7 @@ <x>0</x> <y>0</y> <width>800</width> - <height>30</height> + <height>22</height> </rect> </property> <widget class="QMenu" name="menuFile"> @@ -152,6 +184,7 @@ <property name="title"> <string>Bookmarks</string> </property> + <addaction name="actionShowBookmarksPanel"/> </widget> <widget class="QMenu" name="menuSettings"> <property name="title"> @@ -211,6 +244,17 @@ <string>Close Tab</string> </property> </action> + <action name="actionShowBookmarksPanel"> + <property name="checkable"> + <bool>true</bool> + </property> + <property name="checked"> + <bool>true</bool> + </property> + <property name="text"> + <string>Show Bookmarks Panel</string> + </property> + </action> </widget> <customwidgets> <customwidget> @@ -224,6 +268,11 @@ <header>tabbar/tabbar.h</header> <container>1</container> </customwidget> + <customwidget> + <class>BookmarksPanel</class> + <extends>QTreeView</extends> + <header>panels/bookmarkspanel.hpp</header> + </customwidget> </customwidgets> <resources/> <connections> diff --git a/src/rekonqwindow_class.cpp b/src/rekonqwindow_class.cpp index 037abf9a..c83fd3ae 100644 --- a/src/rekonqwindow_class.cpp +++ b/src/rekonqwindow_class.cpp @@ -28,6 +28,12 @@ RekonqWindow::RekonqWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::Re { ui->setupUi(this); +#ifndef REKONQ_TEST + ui->bookmarksPanel->setModel(Application::instance()->bookmarks()); +#endif + connect(ui->bookmarksPanel, &BookmarksPanel::loadUrl, this, + qOverload<const QUrl &, rekonq::OpenType>(&RekonqWindow::loadUrl)); + connect(ui->tabs, &TabBar::currentChanged, this, [this](RekonqView *view) { if (view == nullptr) { // last tab has been closed close(); @@ -70,6 +76,7 @@ RekonqWindow::RekonqWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::Re // view menu // history menu // bookmarks menu + connect(ui->actionShowBookmarksPanel, &QAction::triggered, this, &RekonqWindow::showBookmarksPanel); // settings menu #ifndef REKONQ_TEST connect(ui->actionSettings, &QAction::triggered, this, @@ -100,3 +107,5 @@ void RekonqWindow::setupShortcuts(RekonqSettings *settings) settings->endGroup(); } + +void RekonqWindow::showBookmarksPanel(bool on) { ui->bookmarksPanel->setVisible(on); } diff --git a/src/test/application_mock.hpp b/src/test/application_mock.hpp index edfaf0bc..7b3af1d1 100644 --- a/src/test/application_mock.hpp +++ b/src/test/application_mock.hpp @@ -11,6 +11,7 @@ public: ~FakeApplication() override = default; MOCK_METHOD(bool, registerPlugin, (const QString &), (override)); + MOCK_METHOD(QAbstractItemModel *, bookmarks, (), (const override)); MOCK_METHOD(RekonqWindow *, newWindow, (), (override)); MOCK_METHOD(RekonqView *, newView, (const QUrl &, RekonqWindow *), (override)); |