Changeset View
Standalone View
src/dolphinmainwindow.cpp
| Show First 20 Lines • Show All 43 Lines • ▼ Show 20 Line(s) | |||||
| 44 | #include "views/viewproperties.h" | 44 | #include "views/viewproperties.h" | ||
| 45 | #include "views/dolphinnewfilemenuobserver.h" | 45 | #include "views/dolphinnewfilemenuobserver.h" | ||
| 46 | #include "dolphin_generalsettings.h" | 46 | #include "dolphin_generalsettings.h" | ||
| 47 | 47 | | |||
| 48 | #include <KActionCollection> | 48 | #include <KActionCollection> | ||
| 49 | #include <KActionMenu> | 49 | #include <KActionMenu> | ||
| 50 | #include <KAuthorized> | 50 | #include <KAuthorized> | ||
| 51 | #include <KConfig> | 51 | #include <KConfig> | ||
| 52 | #include <KConfigGui> | ||||
| 52 | #include <KDualAction> | 53 | #include <KDualAction> | ||
| 53 | #include <KFileItemListProperties> | 54 | #include <KFileItemListProperties> | ||
| 54 | #include <KHelpMenu> | 55 | #include <KHelpMenu> | ||
| 55 | #include <KIO/JobUiDelegate> | 56 | #include <KIO/JobUiDelegate> | ||
| 56 | #include <KIO/OpenFileManagerWindowJob> | 57 | #include <KIO/OpenFileManagerWindowJob> | ||
| 57 | #include <KJobWidgets> | 58 | #include <KJobWidgets> | ||
| 58 | #include <KLocalizedString> | 59 | #include <KLocalizedString> | ||
| 59 | #include <KMessageBox> | 60 | #include <KMessageBox> | ||
| ▲ Show 20 Lines • Show All 135 Lines • ▼ Show 20 Line(s) | 123 | #endif | |||
| 195 | auto *middleClickEventFilter = new MiddleClickActionEventFilter(this); | 196 | auto *middleClickEventFilter = new MiddleClickActionEventFilter(this); | ||
| 196 | connect(middleClickEventFilter, &MiddleClickActionEventFilter::actionMiddleClicked, this, &DolphinMainWindow::slotToolBarActionMiddleClicked); | 197 | connect(middleClickEventFilter, &MiddleClickActionEventFilter::actionMiddleClicked, this, &DolphinMainWindow::slotToolBarActionMiddleClicked); | ||
| 197 | toolBar()->installEventFilter(middleClickEventFilter); | 198 | toolBar()->installEventFilter(middleClickEventFilter); | ||
| 198 | 199 | | |||
| 199 | setupWhatsThis(); | 200 | setupWhatsThis(); | ||
| 200 | } | 201 | } | ||
| 201 | 202 | | |||
| 202 | DolphinMainWindow::~DolphinMainWindow() | 203 | DolphinMainWindow::~DolphinMainWindow() | ||
| 203 | { | 204 | { | ||
broulik: Instead of hardcoding those settings everywhere, please use KConfigXt (the `kcfg` files you can… | |||||
you need to check you're not being loaded as part of a session restore otherwise you'll have two paths call readProperties qApp->isSessionRestored() davidedmundson: you need to check you're not being loaded as part of a session restore otherwise you'll have… | |||||
| 205 | if (GeneralSettings::rememberOpenedTabs()) { | ||||
| 206 | KConfigGui::setSessionConfig(QStringLiteral("dolphin"), QStringLiteral("dolphin")); | ||||
| 207 | KConfig *config = KConfigGui::sessionConfig(); | ||||
| 208 | saveGlobalProperties(config); | ||||
I'm surprised this works at all. AFAICTL it'll never be empty as the instance will find itself and hence won't be empty, unless our DBus interface isn't initialised yet (tbf I don't know when it would be). Either way I don't see why this instance is preferred so I think the code would be clearer if it was simply QString() instead. feverfew: I'm surprised this works at all. AFAICTL it'll never be empty as the instance will find itself… | |||||
It doesn't find itself because Dolphin::dolphinInstanceData() filters out the currently running instance by excludinv entries that match its PID. ngraham: It doesn't find itself because `Dolphin::dolphinInstanceData()` filters out the currently… | |||||
The reason why preferredService was a parameter was for when ShowFolders/ShowItems was called via DBus. It specifically bypasses excluding the running instance as in that case you wanted the running instance to accept those URLs, i.e. marking a sevice as preferred means it is exempt from the check that filters out the currently running instance. In this case you want to see if any other instances (apart from this one) exist, and hence you should be passing QString(). The only reason I can think of why the patch works in this current state is that Q_CLASSINFO macro only initialises the DBus object for the main window after the constructor finishes... feverfew: The reason why `preferredService` was a parameter was for when `ShowFolders`/`ShowItems` was… | |||||
| 209 | savePropertiesInternal(config, 1); | ||||
| 210 | config->sync(); | ||||
| 211 | } | ||||
elvisangelaccio: Please move this logic in `DolphinMainWindow::closeEvent()`. | |||||
| 204 | } | 212 | } | ||
| 205 | 213 | | |||
| 206 | QVector<DolphinViewContainer*> DolphinMainWindow::viewContainers() const | 214 | QVector<DolphinViewContainer*> DolphinMainWindow::viewContainers() const | ||
| 207 | { | 215 | { | ||
| 208 | QVector<DolphinViewContainer*> viewContainers; | 216 | QVector<DolphinViewContainer*> viewContainers; | ||
| 209 | viewContainers.reserve(m_tabWidget->count()); | 217 | viewContainers.reserve(m_tabWidget->count()); | ||
| 210 | for (int i = 0; i < m_tabWidget->count(); ++i) { | 218 | for (int i = 0; i < m_tabWidget->count(); ++i) { | ||
| 211 | viewContainers << m_tabWidget->tabPageAt(i)->activeViewContainer(); | 219 | viewContainers << m_tabWidget->tabPageAt(i)->activeViewContainer(); | ||
| ▲ Show 20 Lines • Show All 2038 Lines • Show Last 20 Lines | |||||
Instead of hardcoding those settings everywhere, please use KConfigXt (the kcfg files you can see for other places in Dolphin), like GeneralSettings