aboutsummaryrefslogtreecommitdiff
path: root/launcher/ui
diff options
context:
space:
mode:
Diffstat (limited to 'launcher/ui')
-rw-r--r--launcher/ui/InstanceWindow.cpp6
-rw-r--r--launcher/ui/MainWindow.cpp30
-rw-r--r--launcher/ui/dialogs/BlockedModsDialog.cpp84
-rw-r--r--launcher/ui/dialogs/BlockedModsDialog.h33
-rw-r--r--launcher/ui/dialogs/BlockedModsDialog.ui38
-rw-r--r--launcher/ui/pages/BasePageContainer.h3
-rw-r--r--launcher/ui/pages/instance/ExternalResourcesPage.cpp10
-rw-r--r--launcher/ui/pages/instance/ExternalResourcesPage.h3
-rw-r--r--launcher/ui/pages/instance/ManagedPackPage.cpp433
-rw-r--r--launcher/ui/pages/instance/ManagedPackPage.h152
-rw-r--r--launcher/ui/pages/instance/ManagedPackPage.ui193
-rw-r--r--launcher/ui/pages/instance/ScreenshotsPage.cpp13
-rw-r--r--launcher/ui/pages/instance/ScreenshotsPage.h7
-rw-r--r--launcher/ui/pages/instance/ServersPage.cpp10
-rw-r--r--launcher/ui/pages/instance/ServersPage.h4
-rw-r--r--launcher/ui/pages/instance/VersionPage.cpp15
-rw-r--r--launcher/ui/pages/instance/VersionPage.h5
-rw-r--r--launcher/ui/pages/instance/WorldListPage.cpp10
-rw-r--r--launcher/ui/pages/instance/WorldListPage.h4
-rw-r--r--launcher/ui/pages/modplatform/flame/FlamePage.cpp23
-rw-r--r--launcher/ui/pages/modplatform/flame/FlamePage.h2
-rw-r--r--launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp6
-rw-r--r--launcher/ui/themes/ITheme.cpp13
-rw-r--r--launcher/ui/widgets/PageContainer.cpp5
-rw-r--r--launcher/ui/widgets/PageContainer.h1
-rw-r--r--launcher/ui/widgets/WideBar.cpp250
-rw-r--r--launcher/ui/widgets/WideBar.h30
27 files changed, 1231 insertions, 152 deletions
diff --git a/launcher/ui/InstanceWindow.cpp b/launcher/ui/InstanceWindow.cpp
index 09ce0d67..c62b370f 100644
--- a/launcher/ui/InstanceWindow.cpp
+++ b/launcher/ui/InstanceWindow.cpp
@@ -132,6 +132,12 @@ InstanceWindow::InstanceWindow(InstancePtr instance, QWidget *parent)
{
connect(m_instance.get(), &BaseInstance::statusChanged, this, &InstanceWindow::on_instanceStatusChanged);
}
+
+ // add ourself as the modpack page's instance window
+ {
+ static_cast<ManagedPackPage*>(m_container->getPage("managed_pack"))->setInstanceWindow(this);
+ }
+
show();
}
diff --git a/launcher/ui/MainWindow.cpp b/launcher/ui/MainWindow.cpp
index 91cc5f29..3651aa15 100644
--- a/launcher/ui/MainWindow.cpp
+++ b/launcher/ui/MainWindow.cpp
@@ -548,8 +548,9 @@ public:
fileMenu->addAction(actionChangeInstGroup);
fileMenu->addAction(actionViewSelectedInstFolder);
fileMenu->addAction(actionExportInstance);
- fileMenu->addAction(actionDeleteInstance);
fileMenu->addAction(actionCopyInstance);
+ fileMenu->addAction(actionDeleteInstance);
+ fileMenu->addAction(actionCreateInstanceShortcut);
fileMenu->addSeparator();
fileMenu->addAction(actionSettings);
@@ -778,8 +779,6 @@ public:
actionCreateInstanceShortcut->setObjectName(QStringLiteral("actionCreateInstanceShortcut"));
actionCreateInstanceShortcut.setTextId(QT_TRANSLATE_NOOP("MainWindow", "Create Shortcut"));
actionCreateInstanceShortcut.setTooltipId(QT_TRANSLATE_NOOP("MainWindow", "Creates a shortcut on your desktop to launch the selected instance."));
- //actionCreateInstanceShortcut->setShortcut(QKeySequence(tr("Ctrl+D"))); // TODO
- // FIXME missing on Legacy, Flat and Flat (White)
actionCreateInstanceShortcut->setIcon(APPLICATION->getThemedIcon("shortcut"));
all_actions.append(&actionCreateInstanceShortcut);
@@ -1684,7 +1683,7 @@ InstanceView
background-image: url(:/backgrounds/%1);
background-attachment: fixed;
background-clip: padding;
- background-position: bottom left;
+ background-position: bottom right;
background-repeat: none;
background-color:palette(base);
})")
@@ -2269,10 +2268,25 @@ void MainWindow::on_actionCreateInstanceShortcut_triggered()
QMessageBox::critical(this, tr("Create instance shortcut"), tr("Failed to create icon for shortcut."));
return;
}
-
- if (FS::createShortcut(FS::PathCombine(desktopPath, m_selectedInstance->name()),
- appPath, { "--launch", m_selectedInstance->id() },
- m_selectedInstance->name(), iconPath)) {
+
+ QString desktopFilePath = FS::PathCombine(desktopPath, m_selectedInstance->name() + ".desktop");
+ QStringList args;
+ if (DesktopServices::isFlatpak()) {
+ QFileDialog fileDialog;
+ // workaround to make sure the portal file dialog opens in the desktop directory
+ fileDialog.setDirectoryUrl(desktopPath);
+ desktopFilePath = fileDialog.getSaveFileName(
+ this, tr("Create Shortcut"), desktopFilePath,
+ tr("Desktop Entries (*.desktop)"));
+ if (desktopFilePath.isEmpty())
+ return; // file dialog canceled by user
+ appPath = "flatpak";
+ QString flatpakAppId = BuildConfig.LAUNCHER_DESKTOPFILENAME;
+ flatpakAppId.remove(".desktop");
+ args.append({ "run", flatpakAppId });
+ }
+ args.append({ "--launch", m_selectedInstance->id() });
+ if (FS::createShortcut(desktopFilePath, appPath, args, m_selectedInstance->name(), iconPath)) {
QMessageBox::information(this, tr("Create instance shortcut"), tr("Created a shortcut to this instance on your desktop!"));
}
else
diff --git a/launcher/ui/dialogs/BlockedModsDialog.cpp b/launcher/ui/dialogs/BlockedModsDialog.cpp
index 214eeeaa..8b49bd1a 100644
--- a/launcher/ui/dialogs/BlockedModsDialog.cpp
+++ b/launcher/ui/dialogs/BlockedModsDialog.cpp
@@ -1,14 +1,41 @@
+// SPDX-FileCopyrightText: 2022 Sefa Eyeoglu <contact@scrumplex.net>
+// SPDX-FileCopyrightText: 2022 Rachel Powers <508861+Ryex@users.noreply.github.com>
+// SPDX-FileCopyrightText: 2022 kumquat-ir <66188216+kumquat-ir@users.noreply.github.com>
+//
+// SPDX-License-Identifier: GPL-3.0-only
+
+/*
+ * Prism Launcher - Minecraft Launcher
+ * Copyright (C) 2022 Sefa Eyeoglu <contact@scrumplex.net>
+ * Copyright (C) 2022 Rachel Powers <508861+Ryex@users.noreply.github.com>
+ * Copyright (C) 2022 kumquat-ir <66188216+kumquat-ir@users.noreply.github.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, version 3.
+ *
+ * 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 <https://www.gnu.org/licenses/>.
+ */
+
#include "BlockedModsDialog.h"
-#include <QDesktopServices>
-#include <QDialogButtonBox>
-#include <QPushButton>
-#include "Application.h"
#include "ui_BlockedModsDialog.h"
+#include "Application.h"
+#include "modplatform/helpers/HashUtils.h"
+
#include <QDebug>
+#include <QDesktopServices>
+#include <QDialogButtonBox>
#include <QDragEnterEvent>
#include <QFileDialog>
#include <QFileInfo>
+#include <QPushButton>
#include <QStandardPaths>
BlockedModsDialog::BlockedModsDialog(QWidget* parent, const QString& title, const QString& text, QList<BlockedMod>& mods)
@@ -19,8 +46,8 @@ BlockedModsDialog::BlockedModsDialog(QWidget* parent, const QString& title, cons
ui->setupUi(this);
- auto openAllButton = ui->buttonBox->addButton(tr("Open All"), QDialogButtonBox::ActionRole);
- connect(openAllButton, &QPushButton::clicked, this, &BlockedModsDialog::openAll);
+ m_openMissingButton = ui->buttonBox->addButton(tr("Open Missing"), QDialogButtonBox::ActionRole);
+ connect(m_openMissingButton, &QPushButton::clicked, this, [this]() { openAll(true); });
auto downloadFolderButton = ui->buttonBox->addButton(tr("Add Download Folder"), QDialogButtonBox::ActionRole);
connect(downloadFolderButton, &QPushButton::clicked, this, &BlockedModsDialog::addDownloadFolder);
@@ -34,15 +61,8 @@ BlockedModsDialog::BlockedModsDialog(QWidget* parent, const QString& title, cons
this->setWindowTitle(title);
ui->labelDescription->setText(text);
- ui->labelExplain->setText(
- QString(tr("Your configured global mods folder and default downloads folder "
- "are automatically checked for the downloaded mods and they will be copied to the instance if found.<br/>"
- "Optionally, you may drag and drop the downloaded mods onto this dialog or add a folder to watch "
- "if you did not download the mods to a default location."))
- .arg(APPLICATION->settings()->get("CentralModsDir").toString(),
- QStandardPaths::writableLocation(QStandardPaths::DownloadLocation)));
-
- // force all URL handeling as external
+
+ // force all URL handling as external
connect(ui->textBrowserWatched, &QTextBrowser::anchorClicked, this, [](const QUrl url) { QDesktopServices::openUrl(url); });
setAcceptDrops(true);
@@ -64,7 +84,15 @@ void BlockedModsDialog::dragEnterEvent(QDragEnterEvent* e)
void BlockedModsDialog::dropEvent(QDropEvent* e)
{
- for (const QUrl& url : e->mimeData()->urls()) {
+ for (QUrl& url : e->mimeData()->urls()) {
+ if (url.scheme().isEmpty()) { // ensure isLocalFile() works correctly
+ url.setScheme("file");
+ }
+
+ if (!url.isLocalFile()) { // can't drop external files here.
+ continue;
+ }
+
QString filePath = url.toLocalFile();
qDebug() << "[Blocked Mods Dialog] Dropped file:" << filePath;
addHashTask(filePath);
@@ -85,10 +113,12 @@ void BlockedModsDialog::done(int r)
disconnect(&m_watcher, &QFileSystemWatcher::directoryChanged, this, &BlockedModsDialog::directoryChanged);
}
-void BlockedModsDialog::openAll()
+void BlockedModsDialog::openAll(bool missingOnly)
{
for (auto& mod : m_mods) {
- QDesktopServices::openUrl(mod.websiteUrl);
+ if (!missingOnly || !mod.matched) {
+ QDesktopServices::openUrl(mod.websiteUrl);
+ }
}
}
@@ -131,8 +161,10 @@ void BlockedModsDialog::update()
if (allModsMatched()) {
ui->labelModsFound->setText("<span style=\"color:green\">✔</span>" + tr("All mods found"));
+ m_openMissingButton->setDisabled(true);
} else {
ui->labelModsFound->setText(tr("Please download the missing mods."));
+ m_openMissingButton->setDisabled(false);
}
}
@@ -243,14 +275,24 @@ void BlockedModsDialog::checkMatchHash(QString hash, QString path)
/// @return boolean: did the path match the name of a blocked mod?
bool BlockedModsDialog::checkValidPath(QString path)
{
- QFileInfo file = QFileInfo(path);
- QString filename = file.fileName();
+ const QFileInfo file = QFileInfo(path);
+ const QString filename = file.fileName();
+ QString laxFilename(filename);
+ laxFilename.replace('+', ' ');
+
+ auto compare = [](QString fsfilename, QString metadataFilename) {
+ return metadataFilename.compare(fsfilename, Qt::CaseInsensitive) == 0;
+ };
for (auto& mod : m_mods) {
- if (mod.name.compare(filename, Qt::CaseInsensitive) == 0) {
+ if (compare(filename, mod.name)) {
qDebug() << "[Blocked Mods Dialog] Name match found:" << mod.name << "| From path:" << path;
return true;
}
+ if (compare(laxFilename, mod.name)) {
+ qDebug() << "[Blocked Mods Dialog] Lax name match found:" << mod.name << "| From path:" << path;
+ return true;
+ }
}
return false;
diff --git a/launcher/ui/dialogs/BlockedModsDialog.h b/launcher/ui/dialogs/BlockedModsDialog.h
index f5aef8bf..014f488a 100644
--- a/launcher/ui/dialogs/BlockedModsDialog.h
+++ b/launcher/ui/dialogs/BlockedModsDialog.h
@@ -1,3 +1,28 @@
+// SPDX-FileCopyrightText: 2022 Sefa Eyeoglu <contact@scrumplex.net>
+// SPDX-FileCopyrightText: 2022 Rachel Powers <508861+Ryex@users.noreply.github.com>
+// SPDX-FileCopyrightText: 2022 kumquat-ir <66188216+kumquat-ir@users.noreply.github.com>
+//
+// SPDX-License-Identifier: GPL-3.0-only
+
+/*
+ * Prism Launcher - Minecraft Launcher
+ * Copyright (C) 2022 Sefa Eyeoglu <contact@scrumplex.net>
+ * Copyright (C) 2022 Rachel Powers <508861+Ryex@users.noreply.github.com>
+ * Copyright (C) 2022 kumquat-ir <66188216+kumquat-ir@users.noreply.github.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, version 3.
+ *
+ * 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 <https://www.gnu.org/licenses/>.
+ */
+
#pragma once
#include <QDialog>
@@ -6,16 +31,17 @@
#include <QFileSystemWatcher>
-#include "modplatform/helpers/HashUtils.h"
-
#include "tasks/ConcurrentTask.h"
+class QPushButton;
+
struct BlockedMod {
QString name;
QString websiteUrl;
QString hash;
bool matched;
QString localPath;
+ QString targetFolder;
};
QT_BEGIN_NAMESPACE
@@ -46,8 +72,9 @@ class BlockedModsDialog : public QDialog {
shared_qobject_ptr<ConcurrentTask> m_hashing_task;
QSet<QString> m_pending_hash_paths;
bool m_rehash_pending;
+ QPushButton* m_openMissingButton;
- void openAll();
+ void openAll(bool missingOnly);
void addDownloadFolder();
void update();
void directoryChanged(QString path);
diff --git a/launcher/ui/dialogs/BlockedModsDialog.ui b/launcher/ui/dialogs/BlockedModsDialog.ui
index 88105178..2292b99c 100644
--- a/launcher/ui/dialogs/BlockedModsDialog.ui
+++ b/launcher/ui/dialogs/BlockedModsDialog.ui
@@ -7,17 +7,23 @@
<x>0</x>
<y>0</y>
<width>400</width>
- <height>455</height>
+ <height>400</height>
</rect>
</property>
+ <property name="minimumSize">
+ <size>
+ <width>0</width>
+ <height>350</height>
+ </size>
+ </property>
<property name="windowTitle">
<string notr="true">BlockedModsDialog</string>
</property>
- <layout class="QVBoxLayout" name="verticalLayout">
+ <layout class="QVBoxLayout" name="verticalLayout" stretch="0,0,3,0,1,0">
<item>
<widget class="QLabel" name="labelDescription">
<property name="text">
- <string notr="true"/>
+ <string notr="true">Placeholder description</string>
</property>
<property name="textFormat">
<enum>Qt::RichText</enum>
@@ -30,7 +36,7 @@
<item>
<widget class="QLabel" name="labelExplain">
<property name="text">
- <string/>
+ <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;Your configured global mods folder and default downloads folder are automatically checked for the downloaded mods and they will be copied to the instance if found.&lt;/p&gt;&lt;p&gt;Optionally, you may drag and drop the downloaded mods onto this dialog or add a folder to watch if you did not download the mods to a default location.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
</property>
<property name="wordWrap">
<bool>true</bool>
@@ -42,12 +48,6 @@
</item>
<item>
<widget class="QTextBrowser" name="textBrowserModsListing">
- <property name="minimumSize">
- <size>
- <width>0</width>
- <height>165</height>
- </size>
- </property>
<property name="acceptRichText">
<bool>true</bool>
</property>
@@ -58,12 +58,6 @@
</item>
<item>
<widget class="QLabel" name="labelWatched">
- <property name="sizePolicy">
- <sizepolicy hsizetype="Preferred" vsizetype="Preferred">
- <horstretch>0</horstretch>
- <verstretch>1</verstretch>
- </sizepolicy>
- </property>
<property name="text">
<string>Watched Folders:</string>
</property>
@@ -71,18 +65,6 @@
</item>
<item>
<widget class="QTextBrowser" name="textBrowserWatched">
- <property name="sizePolicy">
- <sizepolicy hsizetype="Expanding" vsizetype="Minimum">
- <horstretch>0</horstretch>
- <verstretch>0</verstretch>
- </sizepolicy>
- </property>
- <property name="minimumSize">
- <size>
- <width>0</width>
- <height>16</height>
- </size>
- </property>
<property name="baseSize">
<size>
<width>0</width>
diff --git a/launcher/ui/pages/BasePageContainer.h b/launcher/ui/pages/BasePageContainer.h
index f8c7adeb..b41fe12a 100644
--- a/launcher/ui/pages/BasePageContainer.h
+++ b/launcher/ui/pages/BasePageContainer.h
@@ -1,10 +1,13 @@
#pragma once
+class BasePage;
+
class BasePageContainer
{
public:
virtual ~BasePageContainer(){};
virtual bool selectPage(QString pageId) = 0;
+ virtual BasePage* getPage(QString pageId) { return nullptr; };
virtual void refreshContainer() = 0;
virtual bool requestClose() = 0;
};
diff --git a/launcher/ui/pages/instance/ExternalResourcesPage.cpp b/launcher/ui/pages/instance/ExternalResourcesPage.cpp
index 5c919573..c66d1368 100644
--- a/launcher/ui/pages/instance/ExternalResourcesPage.cpp
+++ b/launcher/ui/pages/instance/ExternalResourcesPage.cpp
@@ -67,11 +67,21 @@ void ExternalResourcesPage::ShowContextMenu(const QPoint& pos)
void ExternalResourcesPage::openedImpl()
{
m_model->startWatching();
+
+ auto const setting_name = QString("WideBarVisibility_%1").arg(id());
+ if (!APPLICATION->settings()->contains(setting_name))
+ m_wide_bar_setting = APPLICATION->settings()->registerSetting(setting_name);
+ else
+ m_wide_bar_setting = APPLICATION->settings()->getSetting(setting_name);
+
+ ui->actionsToolbar->setVisibilityState(m_wide_bar_setting->get().toByteArray());
}
void ExternalResourcesPage::closedImpl()
{
m_model->stopWatching();
+
+ m_wide_bar_setting->set(ui->actionsToolbar->getVisibilityState());
}
void ExternalResourcesPage::retranslate()
diff --git a/launcher/ui/pages/instance/ExternalResourcesPage.h b/launcher/ui/pages/instance/ExternalResourcesPage.h
index 11058bf6..2d1a5b51 100644
--- a/launcher/ui/pages/instance/ExternalResourcesPage.h
+++ b/launcher/ui/pages/instance/ExternalResourcesPage.h
@@ -4,6 +4,7 @@
#include <QSortFilterProxyModel>
#include "Application.h"
+#include "settings/Setting.h"
#include "minecraft/MinecraftInstance.h"
#include "ui/pages/BasePage.h"
@@ -70,4 +71,6 @@ class ExternalResourcesPage : public QMainWindow, public BasePage {
QString m_viewFilter;
bool m_controlsEnabled = true;
+
+ std::shared_ptr<Setting> m_wide_bar_setting = nullptr;
};
diff --git a/launcher/ui/pages/instance/ManagedPackPage.cpp b/launcher/ui/pages/instance/ManagedPackPage.cpp
new file mode 100644
index 00000000..4de80468
--- /dev/null
+++ b/launcher/ui/pages/instance/ManagedPackPage.cpp
@@ -0,0 +1,433 @@
+// SPDX-FileCopyrightText: 2022 flow <flowlnlnln@gmail.com>
+//
+// SPDX-License-Identifier: GPL-3.0-only
+
+#include "ManagedPackPage.h"
+#include "ui_ManagedPackPage.h"
+
+#include <QListView>
+#include <QProxyStyle>
+#include <QStyleFactory>
+
+#include <HoeDown.h>
+
+#include "Application.h"
+#include "BuildConfig.h"
+#include "InstanceImportTask.h"
+#include "InstanceList.h"
+#include "InstanceTask.h"
+#include "Json.h"
+
+#include "modplatform/modrinth/ModrinthPackManifest.h"
+
+#include "ui/InstanceWindow.h"
+#include "ui/dialogs/CustomMessageBox.h"
+#include "ui/dialogs/ProgressDialog.h"
+
+/** This is just to override the combo box popup behavior so that the combo box doesn't take the whole screen.
+ * ... thanks Qt.
+ */
+class NoBigComboBoxStyle : public QProxyStyle {
+ Q_OBJECT
+
+ public:
+ NoBigComboBoxStyle(QStyle* style) : QProxyStyle(style) {}
+
+ // clang-format off
+ int styleHint(QStyle::StyleHint hint, const QStyleOption* option = nullptr, const QWidget* widget = nullptr, QStyleHintReturn* returnData = nullptr) const override
+ {
+ if (hint == QStyle::SH_ComboBox_Popup)
+ return false;
+
+ return QProxyStyle::styleHint(hint, option, widget, returnData);
+ }
+ // clang-format on
+};
+
+ManagedPackPage* ManagedPackPage::createPage(BaseInstance* inst, QString type, QWidget* parent)
+{
+ if (type == "modrinth")
+ return new ModrinthManagedPackPage(inst, nullptr, parent);
+ if (type == "flame" && (APPLICATION->capabilities() & Application::SupportsFlame))
+ return new FlameManagedPackPage(inst, nullptr, parent);
+
+ return new GenericManagedPackPage(inst, nullptr, parent);
+}
+
+ManagedPackPage::ManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent)
+ : QWidget(parent), m_instance_window(instance_window), ui(new Ui::ManagedPackPage), m_inst(inst)
+{
+ Q_ASSERT(inst);
+
+ ui->setupUi(this);
+
+ // NOTE: GTK2 themes crash with the proxy style.
+ // This seems like an upstream bug, so there's not much else that can be done.
+ if (!QStyleFactory::keys().contains("gtk2"))
+ ui->versionsComboBox->setStyle(new NoBigComboBoxStyle(ui->versionsComboBox->style()));
+
+ ui->reloadButton->setVisible(false);
+ connect(ui->reloadButton, &QPushButton::clicked, this, [this](bool){
+ ui->reloadButton->setVisible(false);
+
+ m_loaded = false;
+ // Pretend we're opening the page again
+ openedImpl();
+ });
+}
+
+ManagedPackPage::~ManagedPackPage()
+{
+ delete ui;
+}
+
+void ManagedPackPage::openedImpl()
+{
+ ui->packName->setText(m_inst->getManagedPackName());
+ ui->packVersion->setText(m_inst->getManagedPackVersionName());
+ ui->packOrigin->setText(tr("Website: <a href=%1>%2</a> | Pack ID: %3 | Version ID: %4")
+ .arg(url(), displayName(), m_inst->getManagedPackID(), m_inst->getManagedPackVersionID()));
+
+ parseManagedPack();
+}
+
+QString ManagedPackPage::displayName() const
+{
+ auto type = m_inst->getManagedPackType();
+ if (type.isEmpty())
+ return {};
+ if (type == "flame")
+ type = "CurseForge";
+ return type.replace(0, 1, type[0].toUpper());
+}
+
+QIcon ManagedPackPage::icon() const
+{
+ return APPLICATION->getThemedIcon(m_inst->getManagedPackType());
+}
+
+QString ManagedPackPage::helpPage() const
+{
+ return {};
+}
+
+void ManagedPackPage::retranslate()
+{
+ ui->retranslateUi(this);
+}
+
+bool ManagedPackPage::shouldDisplay() const
+{
+ return m_inst->isManagedPack();
+}
+
+bool ManagedPackPage::runUpdateTask(InstanceTask* task)
+{
+ Q_ASSERT(task);
+
+ unique_qobject_ptr<Task> wrapped_task(APPLICATION->instances()->wrapInstanceTask(task));
+
+ connect(task, &Task::failed,
+ [this](QString reason) { CustomMessageBox::selectable(this, tr("Error"), reason, QMessageBox::Critical)->show(); });
+ connect(task, &Task::succeeded, [this, task]() {
+ QStringList warnings = task->warnings();
+ if (warnings.count())
+ CustomMessageBox::selectable(this, tr("Warnings"), warnings.join('\n'), QMessageBox::Warning)->show();
+ });
+ connect(task, &Task::aborted, [this] {
+ CustomMessageBox::selectable(this, tr("Task aborted"), tr("The task has been aborted by the user."), QMessageBox::Information)
+ ->show();
+ });
+
+ ProgressDialog loadDialog(this);
+ loadDialog.setSkipButton(true, tr("Abort"));
+ loadDialog.execWithTask(task);
+
+ return task->wasSuccessful();
+}
+
+void ManagedPackPage::suggestVersion()
+{
+ ui->updateButton->setText(tr("Update pack"));
+ ui->updateButton->setDisabled(false);
+}
+
+void ManagedPackPage::setFailState()
+{
+ qDebug() << "Setting fail state!";
+
+ // We block signals here so that suggestVersion() doesn't get called, causing an assertion fail.
+ ui->versionsComboBox->blockSignals(true);
+ ui->versionsComboBox->clear();
+ ui->versionsComboBox->addItem(tr("Failed to search for available versions."), {});
+ ui->versionsComboBox->blockSignals(false);
+
+ ui->changelogTextBrowser->setText(tr("Failed to request changelog data for this modpack."));
+
+ ui->updateButton->setText(tr("Cannot update!"));
+ ui->updateButton->setDisabled(true);
+
+ ui->reloadButton->setVisible(true);
+}
+
+ModrinthManagedPackPage::ModrinthManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent)
+ : ManagedPackPage(inst, instance_window, parent)
+{
+ Q_ASSERT(inst->isManagedPack());
+ connect(ui->versionsComboBox, SIGNAL(currentIndexChanged(int)), this, SLOT(suggestVersion()));
+ connect(ui->updateButton, &QPushButton::pressed, this, &ModrinthManagedPackPage::update);
+}
+
+// MODRINTH
+
+void ModrinthManagedPackPage::parseManagedPack()
+{
+ qDebug() << "Parsing Modrinth pack";
+
+ // No need for the extra work because we already have everything we need.
+ if (m_loaded)
+ return;
+
+ if (m_fetch_job && m_fetch_job->isRunning())
+ m_fetch_job->abort();
+
+ m_fetch_job.reset(new NetJob(QString("Modrinth::PackVersions(%1)").arg(m_inst->getManagedPackName()), APPLICATION->network()));
+ auto response = std::make_shared<QByteArray>();
+
+ QString id = m_inst->getManagedPackID();
+
+ m_fetch_job->addNetAction(Net::Download::makeByteArray(QString("%1/project/%2/version").arg(BuildConfig.MODRINTH_PROD_URL, id), response.get()));
+
+ QObject::connect(m_fetch_job.get(), &NetJob::succeeded, this, [this, response, id] {
+ QJsonParseError parse_error{};
+ QJsonDocument doc = QJsonDocument::fromJson(*response, &parse_error);
+ if (parse_error.error != QJsonParseError::NoError) {
+ qWarning() << "Error while parsing JSON response from Modrinth at " << parse_error.offset
+ << " reason: " << parse_error.errorString();
+ qWarning() << *response;
+
+ setFailState();
+
+ return;
+ }
+
+ try {
+ Modrinth::loadIndexedVersions(m_pack, doc);
+ } catch (const JSONValidationError& e) {
+ qDebug() << *response;
+ qWarning() << "Error while reading modrinth modpack version: " << e.cause();
+
+ setFailState();
+ return;
+ }
+
+ // We block signals here so that suggestVersion() doesn't get called, causing an assertion fail.
+ ui->versionsComboBox->blockSignals(true);
+ ui->versionsComboBox->clear();
+ ui->versionsComboBox->blockSignals(false);
+
+ for (auto version : m_pack.versions) {
+ QString name = version.version;
+
+ if (!version.name.contains(version.version))
+ name = QString("%1 — %2").arg(version.name, version.version);
+
+ // NOTE: the id from version isn't the same id in the modpack format spec...
+ // e.g. HexMC's 4.4.0 has versionId 4.0.0 in the modpack index..............
+ if (version.version == m_inst->getManagedPackVersionName())
+ name = tr("%1 (Current)").arg(name);
+
+
+ ui->versionsComboBox->addItem(name, QVariant(version.id));
+ }
+
+ suggestVersion();
+
+ m_loaded = true;
+ });
+ QObject::connect(m_fetch_job.get(), &NetJob::failed, this, &ModrinthManagedPackPage::setFailState);
+ QObject::connect(m_fetch_job.get(), &NetJob::aborted, this, &ModrinthManagedPackPage::setFailState);
+
+ ui->changelogTextBrowser->setText(tr("Fetching changelogs..."));
+
+ m_fetch_job->start();
+}
+
+QString ModrinthManagedPackPage::url() const
+{
+ return "https://modrinth.com/mod/" + m_inst->getManagedPackID();
+}
+
+void ModrinthManagedPackPage::suggestVersion()
+{
+ auto index = ui->versionsComboBox->currentIndex();
+ auto version = m_pack.versions.at(index);
+
+ HoeDown md_parser;
+ ui->changelogTextBrowser->setHtml(md_parser.process(version.changelog.toUtf8()));
+
+ ManagedPackPage::suggestVersion();
+}
+
+void ModrinthManagedPackPage::update()
+{
+ auto index = ui->versionsComboBox->currentIndex();
+ auto version = m_pack.versions.at(index);
+
+ QMap<QString, QString> extra_info;
+ // NOTE: Don't use 'm_pack.id' here, since we didn't completely parse all the metadata for the pack, including this field.
+ extra_info.insert("pack_id", m_inst->getManagedPackID());
+ extra_info.insert("pack_version_id", version.id);
+ extra_info.insert("original_instance_id", m_inst->id());
+
+ auto extracted = new InstanceImportTask(version.download_url, this, std::move(extra_info));
+
+ InstanceName inst_name(m_inst->getManagedPackName(), version.version);
+ inst_name.setName(m_inst->name().replace(m_inst->getManagedPackVersionName(), version.version));
+ extracted->setName(inst_name);
+
+ extracted->setGroup(APPLICATION->instances()->getInstanceGroup(m_inst->id()));
+ extracted->setIcon(m_inst->iconKey());
+ extracted->setConfirmUpdate(false);
+
+ auto did_succeed = runUpdateTask(extracted);
+
+ if (m_instance_window && did_succeed)
+ m_instance_window->close();
+}
+
+// FLAME
+
+FlameManagedPackPage::FlameManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent)
+ : ManagedPackPage(inst, instance_window, parent)
+{
+ Q_ASSERT(inst->isManagedPack());
+ connect(ui->versionsComboBox, SIGNAL(currentIndexChanged(int)), this, SLOT(suggestVersion()));
+ connect(ui->updateButton, &QPushButton::pressed, this, &FlameManagedPackPage::update);
+}
+
+void FlameManagedPackPage::parseManagedPack()
+{
+ qDebug() << "Parsing Flame pack";
+
+ // We need to tell the user to redownload the pack, since we didn't save the required info previously
+ if (m_inst->getManagedPackID().isEmpty()) {
+ setFailState();
+ QString message =
+ tr("<h1>Hey there!</h1>"
+ "<h4>"
+ "It seems like your Pack ID is null. This is because of a bug in older versions of the launcher.<br/>"
+ "Unfortunately, we can't do the proper API requests without this information.<br/>"
+ "<br/>"
+ "So, in order for this feature to work, you will need to re-download the modpack from the built-in downloader.<br/>"
+ "<br/>"
+ "Don't worry though, it will ask you to update this instance instead, so you'll not lose this instance!"
+ "</h4>");
+
+ ui->changelogTextBrowser->setHtml(message);
+ return;
+ }
+
+ // No need for the extra work because we already have everything we need.
+ if (m_loaded)
+ return;
+
+ if (m_fetch_job && m_fetch_job->isRunning())
+ m_fetch_job->abort();
+
+ m_fetch_job.reset(new NetJob(QString("Flame::PackVersions(%1)").arg(m_inst->getManagedPackName()), APPLICATION->network()));
+ auto response = std::make_shared<QByteArray>();
+
+ QString id = m_inst->getManagedPackID();
+
+ m_fetch_job->addNetAction(Net::Download::makeByteArray(QString("%1/mods/%2/files").arg(BuildConfig.FLAME_BASE_URL, id), response.get()));
+
+ QObject::connect(m_fetch_job.get(), &NetJob::succeeded, this, [this, response, id] {
+ QJsonParseError parse_error{};
+ QJsonDocument doc = QJsonDocument::fromJson(*response, &parse_error);
+ if (parse_error.error != QJsonParseError::NoError) {
+ qWarning() << "Error while parsing JSON response from Flame at " << parse_error.offset
+ << " reason: " << parse_error.errorString();
+ qWarning() << *response;
+
+ setFailState();
+
+ return;
+ }
+
+ try {
+ auto obj = doc.object();
+ auto data = Json::ensureArray(obj, "data");
+ Flame::loadIndexedPackVersions(m_pack, data);
+ } catch (const JSONValidationError& e) {
+ qDebug() << *response;
+ qWarning() << "Error while reading flame modpack version: " << e.cause();
+
+ setFailState();
+ return;
+ }
+
+ // We block signals here so that suggestVersion() doesn't get called, causing an assertion fail.
+ ui->versionsComboBox->blockSignals(true);
+ ui->versionsComboBox->clear();
+ ui->versionsComboBox->blockSignals(false);
+
+ for (auto version : m_pack.versions) {
+ QString name = version.version;
+
+ if (version.fileId == m_inst->getManagedPackVersionID().toInt())
+ name = tr("%1 (Current)").arg(name);
+
+ ui->versionsComboBox->addItem(name, QVariant(version.fileId));
+ }
+
+ suggestVersion();
+
+ m_loaded = true;
+ });
+ QObject::connect(m_fetch_job.get(), &NetJob::failed, this, &FlameManagedPackPage::setFailState);
+ QObject::connect(m_fetch_job.get(), &NetJob::aborted, this, &FlameManagedPackPage::setFailState);
+
+ m_fetch_job->start();
+}
+
+QString FlameManagedPackPage::url() const
+{
+ // FIXME: We should display the websiteUrl field, but this requires doing the API request first :(
+ return {};
+}
+
+void FlameManagedPackPage::suggestVersion()
+{
+ auto index = ui->versionsComboBox->currentIndex();
+ auto version = m_pack.versions.at(index);
+
+ ui->changelogTextBrowser->setHtml(m_api.getModFileChangelog(m_inst->getManagedPackID().toInt(), version.fileId));
+
+ ManagedPackPage::suggestVersion();
+}
+
+void FlameManagedPackPage::update()
+{
+ auto index = ui->versionsComboBox->currentIndex();
+ auto version = m_pack.versions.at(index);
+
+ QMap<QString, QString> extra_info;
+ extra_info.insert("pack_id", m_inst->getManagedPackID());
+ extra_info.insert("pack_version_id", QString::number(version.fileId));
+ extra_info.insert("original_instance_id", m_inst->id());
+
+ auto extracted = new InstanceImportTask(version.downloadUrl, this, std::move(extra_info));
+
+ extracted->setName(m_inst->name());
+ extracted->setGroup(APPLICATION->instances()->getInstanceGroup(m_inst->id()));
+ extracted->setIcon(m_inst->iconKey());
+ extracted->setConfirmUpdate(false);
+
+ auto did_succeed = runUpdateTask(extracted);
+
+ if (m_instance_window && did_succeed)
+ m_instance_window->close();
+}
+
+#include "ManagedPackPage.moc"
diff --git a/launcher/ui/pages/instance/ManagedPackPage.h b/launcher/ui/pages/instance/ManagedPackPage.h
new file mode 100644
index 00000000..d29a5e88
--- /dev/null
+++ b/launcher/ui/pages/instance/ManagedPackPage.h
@@ -0,0 +1,152 @@
+// SPDX-FileCopyrightText: 2022 flow <flowlnlnln@gmail.com>
+//
+// SPDX-License-Identifier: GPL-3.0-only
+
+#pragma once
+
+#include "BaseInstance.h"
+
+#include "modplatform/modrinth/ModrinthAPI.h"
+#include "modplatform/modrinth/ModrinthPackManifest.h"
+
+#include "modplatform/flame/FlameAPI.h"
+#include "modplatform/flame/FlamePackIndex.h"
+
+#include "ui/pages/BasePage.h"
+
+#include <QWidget>
+
+namespace Ui {
+class ManagedPackPage;
+}
+
+class InstanceTask;
+class InstanceWindow;
+
+class ManagedPackPage : public QWidget, public BasePage {
+ Q_OBJECT
+
+ public:
+ inline static ManagedPackPage* createPage(BaseInstance* inst, QWidget* parent = nullptr)
+ {
+ return ManagedPackPage::createPage(inst, inst->getManagedPackType(), parent);
+ }
+
+ static ManagedPackPage* createPage(BaseInstance* inst, QString type, QWidget* parent = nullptr);
+ ~ManagedPackPage() override;
+
+ [[nodiscard]] QString displayName() const override;
+ [[nodiscard]] QIcon icon() const override;
+ [[nodiscard]] QString helpPage() const override;
+ [[nodiscard]] QString id() const override { return "managed_pack"; }
+ [[nodiscard]] bool shouldDisplay() const override;
+
+ void openedImpl() override;
+
+ bool apply() override { return true; }
+ void retranslate() override;
+
+ /** Gets the necessary information about the managed pack, such as
+ * available versions*/
+ virtual void parseManagedPack(){};
+
+ /** URL of the managed pack.
+ * Not the version-specific one.
+ */
+ [[nodiscard]] virtual QString url() const { return {}; };
+
+ void setInstanceWindow(InstanceWindow* window) { m_instance_window = window; }
+
+ public slots:
+ /** Gets the current version selection and update the UI, including the update button and the changelog.
+ */
+ virtual void suggestVersion();
+
+ virtual void update(){};
+
+ protected slots:
+ /** Does the necessary UI changes for when something failed.
+ *
+ * This includes:
+ * - Setting an appropriate text on the version selector to indicate a fail;
+ * - Setting an appropriate text on the changelog text browser to indicate a fail;
+ * - Disable the update button.
+ */
+ void setFailState();
+
+ protected:
+ ManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent = nullptr);
+
+ /** Run the InstanceTask, with a progress dialog and all.
+ * Similar to MainWindow::instanceFromInstanceTask
+ *
+ * Returns whether the task was successful.
+ */
+ bool runUpdateTask(InstanceTask*);
+
+ protected:
+ InstanceWindow* m_instance_window = nullptr;
+
+ Ui::ManagedPackPage* ui;
+ BaseInstance* m_inst;
+
+ bool m_loaded = false;
+};
+
+/** Simple page for when we aren't a managed pack. */
+class GenericManagedPackPage final : public ManagedPackPage {
+ Q_OBJECT
+
+ public:
+ GenericManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent = nullptr)
+ : ManagedPackPage(inst, instance_window, parent)
+ {}
+ ~GenericManagedPackPage() override = default;
+
+ // TODO: We may want to show this page with some useful info at some point.
+ [[nodiscard]] bool shouldDisplay() const override { return false; };
+};
+
+class ModrinthManagedPackPage final : public ManagedPackPage {
+ Q_OBJECT
+
+ public:
+ ModrinthManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent = nullptr);
+ ~ModrinthManagedPackPage() override = default;
+
+ void parseManagedPack() override;
+ [[nodiscard]] QString url() const override;
+
+ public slots:
+ void suggestVersion() override;
+
+ void update() override;
+
+ private:
+ NetJob::Ptr m_fetch_job = nullptr;
+
+ Modrinth::Modpack m_pack;
+ ModrinthAPI m_api;
+};
+
+class FlameManagedPackPage final : public ManagedPackPage {
+ Q_OBJECT
+
+ public:
+ FlameManagedPackPage(BaseInstance* inst, InstanceWindow* instance_window, QWidget* parent = nullptr);
+ ~FlameManagedPackPage() override = default;
+
+ void parseManagedPack() override;
+ [[nodiscard]] QString url() const override;
+
+ public slots:
+ void suggestVersion() override;
+
+ void update() override;
+
+ private:
+ NetJob::Ptr m_fetch_job = nullptr;
+
+ Flame::IndexedPack m_pack;
+ FlameAPI m_api;
+};
diff --git a/launcher/ui/pages/instance/ManagedPackPage.ui b/launcher/ui/pages/instance/ManagedPackPage.ui
new file mode 100644
index 00000000..bbe44a94
--- /dev/null
+++ b/launcher/ui/pages/instance/ManagedPackPage.ui
@@ -0,0 +1,193 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
+ <class>ManagedPackPage</class>
+ <widget class="QWidget" name="ManagedPackPage">
+ <property name="geometry">
+ <rect>
+ <x>0</x>
+ <y>0</y>
+ <width>731</width>
+ <height>538</height>
+ </rect>
+ </property>
+ <layout class="QGridLayout" name="gridLayout">
+ <property name="leftMargin">
+ <number>9</number>
+ </property>
+ <property name="topMargin">
+ <number>9</number>
+ </property>
+ <property name="rightMargin">
+ <number>9</number>
+ </property>
+ <property name="bottomMargin">
+ <number>9</number>
+ </property>
+ <item row="0" column="0">
+ <layout class="QVBoxLayout" name="verticalLayout_2">
+ <item>
+ <widget class="QGroupBox" name="packInformationBox">
+ <property name="sizePolicy">
+ <sizepolicy hsizetype="Preferred" vsizetype="Fixed">
+ <horstretch>0</horstretch>
+ <verstretch>0</verstretch>
+ </sizepolicy>
+ </property>
+ <property name="title">
+ <string>Pack information</string>
+ </property>
+ <layout class="QFormLayout" name="formLayout_2">
+ <item row="0" column="0">
+ <layout class="QHBoxLayout" name="packNameLayout">
+ <item>
+ <widget class="QLabel" name="packNameLabel">
+ <property name="text">
+ <string>Pack name:</string>
+ </property>
+ </widget>
+ </item>
+ <item>
+ <widget class="QLabel" name="packName">
+ <property name="text">
+ <string notr="true">placeholder</string>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </item>
+ <item row="1" column="0">
+ <layout class="QHBoxLayout" name="packVersionLayout">
+ <item>
+ <widget class="QLabel" name="packVersionLabel">
+ <property name="text">
+ <string>Current version:</string>
+ </property>
+ </widget>
+ </item>
+ <item>
+ <widget class="QLabel" name="packVersion">
+ <property name="cursor">
+ <cursorShape>IBeamCursor</cursorShape>
+ </property>
+ <property name="text">
+ <string notr="true">placeholder</string>
+ </property>
+ <property name="textInteractionFlags">
+ <set>Qt::LinksAccessibleByMouse|Qt::TextSelectableByMouse</set>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </item>
+ <item row="2" column="0">
+ <layout class="QHBoxLayout" name="packOriginLayout">
+ <item>
+ <widget class="QLabel" name="packOriginLabel">
+ <property name="text">
+ <string>Provider information:</string>
+ </property>
+ </widget>
+ </item>
+ <item>
+ <widget class="QLabel" name="packOrigin">
+ <property name="cursor">
+ <cursorShape>IBeamCursor</cursorShape>
+ </property>
+ <property name="text">
+ <string notr="true">placeholder</string>
+ </property>
+ <property name="textFormat">
+ <enum>Qt::RichText</enum>
+ </property>
+ <property name="openExternalLinks">
+ <bool>true</bool>
+ </property>
+ <property name="textInteractionFlags">
+ <set>Qt::LinksAccessibleByMouse|Qt::TextSelectableByMouse</set>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </item>
+ </layout>
+ </widget>
+ </item>
+ <item>
+ <widget class="Line" name="line">
+ <property name="orientation">
+ <enum>Qt::Horizontal</enum>
+ </property>
+ </widget>
+ </item>
+ <item>
+ <layout class="QHBoxLayout" name="horizontalLayout">
+ <item>
+ <widget class="QLabel" name="updateToVersionLabel">
+ <property name="sizePolicy">
+ <sizepolicy hsizetype="Fixed" vsizetype="Preferred">
+ <horstretch>0</horstretch>
+ <verstretch>0</verstretch>
+ </sizepolicy>
+ </property>
+ <property name="text">
+ <string>Update to version:</string>
+ </property>
+ </widget>
+ </item>
+ <item>
+ <widget class="QComboBox" name="versionsComboBox"/>
+ </item>
+ <item>
+ <widget class="QPushButton" name="updateButton">
+ <property name="enabled">
+ <bool>false</bool>
+ </property>
+ <property name="sizePolicy">
+ <sizepolicy hsizetype="Fixed" vsizetype="Fixed">
+ <horstretch>0</horstretch>
+ <verstretch>0</verstretch>
+ </sizepolicy>
+ </property>
+ <property name="text">
+ <string>Fetching versions...</string>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </item>
+ <item>
+ <widget class="QGroupBox" name="changelogBox">
+ <property name="sizePolicy">
+ <sizepolicy hsizetype="Preferred" vsizetype="Expanding">
+ <horstretch>0</horstretch>
+ <verstretch>0</verstretch>
+ </sizepolicy>
+ </property>
+ <property name="title">
+ <string>Changelog</string>
+ </property>
+ <layout class="QVBoxLayout" name="verticalLayout_3">
+ <item>
+ <widget class="QTextBrowser" name="changelogTextBrowser">
+ <property name="placeholderText">
+ <string>No changelog available for this version!</string>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </widget>
+ </item>
+ </layout>
+ </item>
+ <item row="1" column="0">
+ <widget class="QPushButton" name="reloadButton">
+ <property name="text">
+ <string>Reload page</string>
+ </property>
+ </widget>
+ </item>
+ </layout>
+ </widget>
+ <resources/>
+ <connections/>
+</ui>
diff --git a/launcher/ui/pages/instance/ScreenshotsPage.cpp b/launcher/ui/pages/instance/ScreenshotsPage.cpp
index c97253e4..0092aef3 100644
--- a/launcher/ui/pages/instance/ScreenshotsPage.cpp
+++ b/launcher/ui/pages/instance/ScreenshotsPage.cpp
@@ -537,6 +537,19 @@ void ScreenshotsPage::openedImpl()
ui->listView->setModel(nullptr);
}
}
+
+ auto const setting_name = QString("WideBarVisibility_%1").arg(id());
+ if (!APPLICATION->settings()->contains(setting_name))
+ m_wide_bar_setting = APPLICATION->settings()->registerSetting(setting_name);
+ else
+ m_wide_bar_setting = APPLICATION->settings()->getSetting(setting_name);
+
+ ui->toolBar->setVisibilityState(m_wide_bar_setting->get().toByteArray());
+}
+
+void ScreenshotsPage::closedImpl()
+{
+ m_wide_bar_setting->set(ui->toolBar->getVisibilityState());
}
#include "ScreenshotsPage.moc"
diff --git a/launcher/ui/pages/instance/ScreenshotsPage.h b/launcher/ui/pages/instance/ScreenshotsPage.h
index c22706af..2eb0de04 100644
--- a/launcher/ui/pages/instance/ScreenshotsPage.h
+++ b/launcher/ui/pages/instance/ScreenshotsPage.h
@@ -40,6 +40,8 @@
#include "ui/pages/BasePage.h"
#include <Application.h>
+#include "settings/Setting.h"
+
class QFileSystemModel;
class QIdentityProxyModel;
namespace Ui
@@ -59,7 +61,8 @@ public:
explicit ScreenshotsPage(QString path, QWidget *parent = 0);
virtual ~ScreenshotsPage();
- virtual void openedImpl() override;
+ void openedImpl() override;
+ void closedImpl() override;
enum
{
@@ -110,4 +113,6 @@ private:
QString m_folder;
bool m_valid = false;
bool m_uploadActive = false;
+
+ std::shared_ptr<Setting> m_wide_bar_setting = nullptr;
};
diff --git a/launcher/ui/pages/instance/ServersPage.cpp b/launcher/ui/pages/instance/ServersPage.cpp
index d64bcb76..a625e20b 100644
--- a/launcher/ui/pages/instance/ServersPage.cpp
+++ b/launcher/ui/pages/instance/ServersPage.cpp
@@ -765,11 +765,21 @@ void ServersPage::updateState()
void ServersPage::openedImpl()
{
m_model->observe();
+
+ auto const setting_name = QString("WideBarVisibility_%1").arg(id());
+ if (!APPLICATION->settings()->contains(setting_name))
+ m_wide_bar_setting = APPLICATION->settings()->registerSetting(setting_name);
+ else
+ m_wide_bar_setting = APPLICATION->settings()->getSetting(setting_name);
+
+ ui->toolBar->setVisibilityState(m_wide_bar_setting->get().toByteArray());
}
void ServersPage::closedImpl()
{
m_model->unobserve();
+
+ m_wide_bar_setting->set(ui->toolBar->getVisibilityState());
}
void ServersPage::on_actionAdd_triggered()
diff --git a/launcher/ui/pages/instance/ServersPage.h b/launcher/ui/pages/instance/ServersPage.h
index ee63353a..476e7d70 100644
--- a/launcher/ui/pages/instance/ServersPage.h
+++ b/launcher/ui/pages/instance/ServersPage.h
@@ -42,6 +42,8 @@
#include "ui/pages/BasePage.h"
#include <Application.h>
+#include "settings/Setting.h"
+
namespace Ui
{
class ServersPage;
@@ -112,5 +114,7 @@ private: // data
Ui::ServersPage *ui = nullptr;
ServersModel * m_model = nullptr;
InstancePtr m_inst = nullptr;
+
+ std::shared_ptr<Setting> m_wide_bar_setting = nullptr;
};
diff --git a/launcher/ui/pages/instance/VersionPage.cpp b/launcher/ui/pages/instance/VersionPage.cpp
index 7f98cba2..c8a65f10 100644
--- a/launcher/ui/pages/instance/VersionPage.cpp
+++ b/launcher/ui/pages/instance/VersionPage.cpp
@@ -126,6 +126,21 @@ void VersionPage::retranslate()
ui->retranslateUi(this);
}
+void VersionPage::openedImpl()
+{
+ auto const setting_name = QString("WideBarVisibility_%1").arg(id());
+ if (!APPLICATION->settings()->contains(setting_name))
+ m_wide_bar_setting = APPLICATION->settings()->registerSetting(setting_name);
+ else
+ m_wide_bar_setting = APPLICATION->settings()->getSetting(setting_name);
+
+ ui->toolBar->setVisibilityState(m_wide_bar_setting->get().toByteArray());
+}
+void VersionPage::closedImpl()
+{
+ m_wide_bar_setting->set(ui->toolBar->getVisibilityState());
+}
+
QMenu * VersionPage::createPopupMenu()
{
QMenu* filteredMenu = QMainWindow::createPopupMenu();
diff --git a/launcher/ui/pages/instance/VersionPage.h b/launcher/ui/pages/instance/VersionPage.h
index 23d2a1b3..166f36bb 100644
--- a/launcher/ui/pages/instance/VersionPage.h
+++ b/launcher/ui/pages/instance/VersionPage.h
@@ -70,6 +70,9 @@ public:
virtual bool shouldDisplay() const override;
void retranslate() override;
+ void openedImpl() override;
+ void closedImpl() override;
+
private slots:
void on_actionChange_version_triggered();
void on_actionInstall_Forge_triggered();
@@ -116,6 +119,8 @@ private:
int currentIdx = 0;
bool controlsEnabled = false;
+ std::shared_ptr<Setting> m_wide_bar_setting = nullptr;
+
public slots:
void versionCurrent(const QModelIndex &current, const QModelIndex &previous);
diff --git a/launcher/ui/pages/instance/WorldListPage.cpp b/launcher/ui/pages/instance/WorldListPage.cpp
index 85cc01ff..93458ce4 100644
--- a/launcher/ui/pages/instance/WorldListPage.cpp
+++ b/launcher/ui/pages/instance/WorldListPage.cpp
@@ -113,11 +113,21 @@ WorldListPage::WorldListPage(BaseInstance *inst, std::shared_ptr<WorldList> worl
void WorldListPage::openedImpl()
{
m_worlds->startWatching();
+
+ auto const setting_name = QString("WideBarVisibility_%1").arg(id());
+ if (!APPLICATION->settings()->contains(setting_name))
+ m_wide_bar_setting = APPLICATION->settings()->registerSetting(setting_name);
+ else
+ m_wide_bar_setting = APPLICATION->settings()->getSetting(setting_name);
+
+ ui->toolBar->setVisibilityState(m_wide_bar_setting->get().toByteArray());
}
void WorldListPage::closedImpl()
{
m_worlds->stopWatching();
+
+ m_wide_bar_setting->set(ui->toolBar->getVisibilityState());
}
WorldListPage::~WorldListPage()
diff --git a/launcher/ui/pages/instance/WorldListPage.h b/launcher/ui/pages/instance/WorldListPage.h
index 1dc9e53e..925521be 100644
--- a/launcher/ui/pages/instance/WorldListPage.h
+++ b/launcher/ui/pages/instance/WorldListPage.h
@@ -42,6 +42,8 @@
#include <Application.h>
#include <LoggedProcess.h>
+#include "settings/Setting.h"
+
class WorldList;
namespace Ui
{
@@ -102,6 +104,8 @@ private:
unique_qobject_ptr<LoggedProcess> m_mceditProcess;
bool m_mceditStarting = false;
+ std::shared_ptr<Setting> m_wide_bar_setting = nullptr;
+
private slots:
void on_actionCopy_Seed_triggered();
void on_actionMCEdit_triggered();
diff --git a/launcher/ui/pages/modplatform/flame/FlamePage.cpp b/launcher/ui/pages/modplatform/flame/FlamePage.cpp
index a65b6585..f9ac4a78 100644
--- a/launcher/ui/pages/modplatform/flame/FlamePage.cpp
+++ b/launcher/ui/pages/modplatform/flame/FlamePage.cpp
@@ -197,12 +197,18 @@ void FlamePage::suggestCurrent()
return;
}
- if (selectedVersion.isEmpty() || selectedVersion == "-1") {
+ if (m_selected_version_index == -1) {
dialog->setSuggestedPack();
return;
}
- dialog->setSuggestedPack(current.name, new InstanceImportTask(selectedVersion,this));
+ auto version = current.versions.at(m_selected_version_index);
+
+ QMap<QString, QString> extra_info;
+ extra_info.insert("pack_id", QString::number(current.addonId));
+ extra_info.insert("pack_version_id", QString::number(version.fileId));
+
+ dialog->setSuggestedPack(current.name, new InstanceImportTask(version.downloadUrl, this, std::move(extra_info)));
QString editedLogoName;
editedLogoName = "curseforge_" + current.logoName.section(".", 0, 0);
listModel->getLogo(current.logoName, current.logoUrl,
@@ -211,11 +217,18 @@ void FlamePage::suggestCurrent()
void FlamePage::onVersionSelectionChanged(QString data)
{
- if (data.isNull() || data.isEmpty()) {
- selectedVersion = "";
+ bool is_blocked = false;
+ ui->versionSelectionBox->currentData().toInt(&is_blocked);
+
+ if (data.isNull() || data.isEmpty() || is_blocked) {
+ m_selected_version_index = -1;
return;
}
- selectedVersion = ui->versionSelectionBox->currentData().toString();
+
+ m_selected_version_index = ui->versionSelectionBox->currentIndex();
+
+ Q_ASSERT(current.versions.at(m_selected_version_index).downloadUrl == ui->versionSelectionBox->currentData().toString());
+
suggestCurrent();
}
diff --git a/launcher/ui/pages/modplatform/flame/FlamePage.h b/launcher/ui/pages/modplatform/flame/FlamePage.h
index 8130e416..8bdca38e 100644
--- a/launcher/ui/pages/modplatform/flame/FlamePage.h
+++ b/launcher/ui/pages/modplatform/flame/FlamePage.h
@@ -99,5 +99,5 @@ private:
Flame::ListModel* listModel = nullptr;
Flame::IndexedPack current;
- QString selectedVersion;
+ int m_selected_version_index = -1;
};
diff --git a/launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp b/launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp
index 4482774c..8ab2ad1d 100644
--- a/launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp
+++ b/launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp
@@ -300,7 +300,11 @@ void ModrinthPage::suggestCurrent()
for (auto& ver : current.versions) {
if (ver.id == selectedVersion) {
- dialog->setSuggestedPack(current.name, ver.version, new InstanceImportTask(ver.download_url, this));
+ QMap<QString, QString> extra_info;
+ extra_info.insert("pack_id", current.id);
+ extra_info.insert("pack_version_id", ver.id);
+
+ dialog->setSuggestedPack(current.name, ver.version, new InstanceImportTask(ver.download_url, this, std::move(extra_info)));
auto iconName = current.iconName;
m_model->getLogo(iconName, current.iconUrl.toString(),
[this, iconName](QString logo) { dialog->setSuggestedIconFromFile(logo, iconName); });
diff --git a/launcher/ui/themes/ITheme.cpp b/launcher/ui/themes/ITheme.cpp
index 7247b444..8bfc466d 100644
--- a/launcher/ui/themes/ITheme.cpp
+++ b/launcher/ui/themes/ITheme.cpp
@@ -6,19 +6,14 @@
void ITheme::apply(bool)
{
+ APPLICATION->setStyleSheet(QString());
QApplication::setStyle(QStyleFactory::create(qtTheme()));
- if(hasColorScheme())
- {
+ if (hasColorScheme()) {
QApplication::setPalette(colorScheme());
}
- if(hasStyleSheet())
- {
+ if (hasStyleSheet())
APPLICATION->setStyleSheet(appStyleSheet());
- }
- else
- {
- APPLICATION->setStyleSheet(QString());
- }
+
QDir::setSearchPaths("theme", searchPaths());
}
diff --git a/launcher/ui/widgets/PageContainer.cpp b/launcher/ui/widgets/PageContainer.cpp
index 8d606820..0a06a351 100644
--- a/launcher/ui/widgets/PageContainer.cpp
+++ b/launcher/ui/widgets/PageContainer.cpp
@@ -130,6 +130,11 @@ bool PageContainer::selectPage(QString pageId)
return false;
}
+BasePage* PageContainer::getPage(QString pageId)
+{
+ return m_model->findPageEntryById(pageId);
+}
+
void PageContainer::refreshContainer()
{
m_proxyModel->invalidate();
diff --git a/launcher/ui/widgets/PageContainer.h b/launcher/ui/widgets/PageContainer.h
index 80d87a9b..97e294dc 100644
--- a/launcher/ui/widgets/PageContainer.h
+++ b/launcher/ui/widgets/PageContainer.h
@@ -79,6 +79,7 @@ public:
}
virtual bool selectPage(QString pageId) override;
+ BasePage* getPage(QString pageId) override;
void refreshContainer() override;
virtual void setParentContainer(BasePageContainer * container)
diff --git a/launcher/ui/widgets/WideBar.cpp b/launcher/ui/widgets/WideBar.cpp
index 79f1e0c9..428be563 100644
--- a/launcher/ui/widgets/WideBar.cpp
+++ b/launcher/ui/widgets/WideBar.cpp
@@ -1,19 +1,24 @@
#include "WideBar.h"
+
+#include <QContextMenuEvent>
+#include <QCryptographicHash>
#include <QToolButton>
-#include <QMenu>
-class ActionButton : public QToolButton
-{
+class ActionButton : public QToolButton {
Q_OBJECT
-public:
- ActionButton(QAction * action, QWidget * parent = 0) : QToolButton(parent), m_action(action) {
+ public:
+ ActionButton(QAction* action, QWidget* parent = nullptr) : QToolButton(parent), m_action(action)
+ {
setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
+
connect(action, &QAction::changed, this, &ActionButton::actionChanged);
connect(this, &ActionButton::clicked, action, &QAction::trigger);
+
actionChanged();
};
-private slots:
- void actionChanged() {
+ public slots:
+ void actionChanged()
+ {
setEnabled(m_action->isEnabled());
setChecked(m_action->isChecked());
setCheckable(m_action->isCheckable());
@@ -23,137 +28,242 @@ private slots:
setHidden(!m_action->isVisible());
setFocusPolicy(Qt::NoFocus);
}
-private:
- QAction * m_action;
-};
+ private:
+ QAction* m_action;
+};
WideBar::WideBar(const QString& title, QWidget* parent) : QToolBar(title, parent)
{
setFloatable(false);
setMovable(false);
+
+ setContextMenuPolicy(Qt::ContextMenuPolicy::CustomContextMenu);
+ connect(this, &QToolBar::customContextMenuRequested, this, &WideBar::showVisibilityMenu);
}
WideBar::WideBar(QWidget* parent) : QToolBar(parent)
{
setFloatable(false);
setMovable(false);
-}
-
-struct WideBar::BarEntry {
- enum Type {
- None,
- Action,
- Separator,
- Spacer
- } type = None;
- QAction *qAction = nullptr;
- QAction *wideAction = nullptr;
-};
-
-WideBar::~WideBar()
-{
- for(auto *iter: m_entries) {
- delete iter;
- }
+ setContextMenuPolicy(Qt::ContextMenuPolicy::CustomContextMenu);
+ connect(this, &QToolBar::customContextMenuRequested, this, &WideBar::showVisibilityMenu);
}
void WideBar::addAction(QAction* action)
{
- auto entry = new BarEntry();
- entry->qAction = addWidget(new ActionButton(action, this));
- entry->wideAction = action;
- entry->type = BarEntry::Action;
+ BarEntry entry;
+ entry.bar_action = addWidget(new ActionButton(action, this));
+ entry.menu_action = action;
+ entry.type = BarEntry::Type::Action;
+
m_entries.push_back(entry);
+
+ m_menu_state = MenuState::Dirty;
}
void WideBar::addSeparator()
{
- auto entry = new BarEntry();
- entry->qAction = QToolBar::addSeparator();
- entry->type = BarEntry::Separator;
+ BarEntry entry;
+ entry.bar_action = QToolBar::addSeparator();
+ entry.type = BarEntry::Type::Separator;
+
m_entries.push_back(entry);
}
-auto WideBar::getMatching(QAction* act) -> QList<BarEntry*>::iterator
+auto WideBar::getMatching(QAction* act) -> QList<BarEntry>::iterator
{
- auto iter = std::find_if(m_entries.begin(), m_entries.end(), [act](BarEntry * entry) {
- return entry->wideAction == act;
- });
-
+ auto iter = std::find_if(m_entries.begin(), m_entries.end(), [act](BarEntry const& entry) { return entry.menu_action == act; });
+
return iter;
}
-void WideBar::insertActionBefore(QAction* before, QAction* action){
+void WideBar::insertActionBefore(QAction* before, QAction* action)
+{
auto iter = getMatching(before);
- if(iter == m_entries.end())
+ if (iter == m_entries.end())
return;
- auto entry = new BarEntry();
- entry->qAction = insertWidget((*iter)->qAction, new ActionButton(action, this));
- entry->wideAction = action;
- entry->type = BarEntry::Action;
+ BarEntry entry;
+ entry.bar_action = insertWidget(iter->bar_action, new ActionButton(action, this));
+ entry.menu_action = action;
+ entry.type = BarEntry::Type::Action;
+
m_entries.insert(iter, entry);
+
+ m_menu_state = MenuState::Dirty;
}
-void WideBar::insertActionAfter(QAction* after, QAction* action){
+void WideBar::insertActionAfter(QAction* after, QAction* action)
+{
auto iter = getMatching(after);
- if(iter == m_entries.end())
+ if (iter == m_entries.end())
return;
- auto entry = new BarEntry();
- entry->qAction = insertWidget((*(iter+1))->qAction, new ActionButton(action, this));
- entry->wideAction = action;
- entry->type = BarEntry::Action;
+ BarEntry entry;
+ entry.bar_action = insertWidget((iter + 1)->bar_action, new ActionButton(action, this));
+ entry.menu_action = action;
+ entry.type = BarEntry::Type::Action;
+
m_entries.insert(iter + 1, entry);
+
+ m_menu_state = MenuState::Dirty;
}
void WideBar::insertSpacer(QAction* action)
{
auto iter = getMatching(action);
- if(iter == m_entries.end())
+ if (iter == m_entries.end())
return;
- QWidget* spacer = new QWidget();
+ auto* spacer = new QWidget();
spacer->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);
- auto entry = new BarEntry();
- entry->qAction = insertWidget((*iter)->qAction, spacer);
- entry->type = BarEntry::Spacer;
+ BarEntry entry;
+ entry.bar_action = insertWidget(iter->bar_action, spacer);
+ entry.type = BarEntry::Type::Spacer;
m_entries.insert(iter, entry);
}
void WideBar::insertSeparator(QAction* before)
{
auto iter = getMatching(before);
- if(iter == m_entries.end())
+ if (iter == m_entries.end())
return;
- auto entry = new BarEntry();
- entry->qAction = QToolBar::insertSeparator(before);
- entry->type = BarEntry::Separator;
+ BarEntry entry;
+ entry.bar_action = QToolBar::insertSeparator(before);
+ entry.type = BarEntry::Type::Separator;
+
m_entries.insert(iter, entry);
}
-QMenu * WideBar::createContextMenu(QWidget *parent, const QString & title)
+QMenu* WideBar::createContextMenu(QWidget* parent, const QString& title)
{
- QMenu *contextMenu = new QMenu(title, parent);
- for(auto & item: m_entries) {
- switch(item->type) {
+ auto* contextMenu = new QMenu(title, parent);
+ for (auto& item : m_entries) {
+ switch (item.type) {
default:
- case BarEntry::None:
+ case BarEntry::Type::None:
break;
- case BarEntry::Separator:
- case BarEntry::Spacer:
+ case BarEntry::Type::Separator:
+ case BarEntry::Type::Spacer:
contextMenu->addSeparator();
break;
- case BarEntry::Action:
- contextMenu->addAction(item->wideAction);
+ case BarEntry::Type::Action:
+ contextMenu->addAction(item.menu_action);
break;
}
}
return contextMenu;
}
+static void copyAction(QAction* from, QAction* to)
+{
+ Q_ASSERT(from);
+ Q_ASSERT(to);
+
+ to->setText(from->text());
+ to->setIcon(from->icon());
+ to->setToolTip(from->toolTip());
+}
+
+void WideBar::showVisibilityMenu(QPoint const& position)
+{
+ if (!m_bar_menu)
+ m_bar_menu = std::make_unique<QMenu>(this);
+
+ if (m_menu_state == MenuState::Dirty) {
+ for (auto* old_action : m_bar_menu->actions())
+ old_action->deleteLater();
+
+ m_bar_menu->clear();
+
+ for (auto& entry : m_entries) {
+ if (entry.type != BarEntry::Type::Action)
+ continue;
+
+ auto act = new QAction();
+ copyAction(entry.menu_action, act);
+
+ act->setCheckable(true);
+ act->setChecked(entry.bar_action->isVisible());
+
+ connect(act, &QAction::toggled, entry.bar_action, [this, &entry](bool toggled){
+ entry.bar_action->setVisible(toggled);
+
+ // NOTE: This is needed so that disabled actions get reflected on the button when it is made visible.
+ static_cast<ActionButton*>(widgetForAction(entry.bar_action))->actionChanged();
+ });
+
+ m_bar_menu->addAction(act);
+ }
+
+ m_menu_state = MenuState::Fresh;
+ }
+
+ m_bar_menu->popup(mapToGlobal(position));
+}
+
+[[nodiscard]] QByteArray WideBar::getVisibilityState() const
+{
+ QByteArray state;
+
+ for (auto const& entry : m_entries) {
+ if (entry.type != BarEntry::Type::Action)
+ continue;
+
+ state.append(entry.bar_action->isVisible() ? '1' : '0');
+ }
+
+ state.append(',');
+ state.append(getHash());
+
+ return state;
+}
+
+void WideBar::setVisibilityState(QByteArray&& state)
+{
+ auto split = state.split(',');
+
+ auto bits = split.first();
+ auto hash = split.last();
+
+ // If the actions changed, we better not try to load the old one to avoid unwanted hiding
+ if (!checkHash(hash))
+ return;
+
+ qsizetype i = 0;
+ for (auto& entry : m_entries) {
+ if (entry.type != BarEntry::Type::Action)
+ continue;
+ if (i == bits.size())
+ break;
+
+ entry.bar_action->setVisible(bits.at(i++) == '1');
+
+ // NOTE: This is needed so that disabled actions get reflected on the button when it is made visible.
+ static_cast<ActionButton*>(widgetForAction(entry.bar_action))->actionChanged();
+ }
+}
+
+QByteArray WideBar::getHash() const
+{
+ QCryptographicHash hash(QCryptographicHash::Sha1);
+ for (auto const& entry : m_entries) {
+ if (entry.type != BarEntry::Type::Action)
+ continue;
+ hash.addData(entry.menu_action->text().toLatin1());
+ }
+
+ return hash.result().toBase64();
+}
+
+bool WideBar::checkHash(QByteArray const& old_hash) const
+{
+ return old_hash == getHash();
+}
+
+
#include "WideBar.moc"
diff --git a/launcher/ui/widgets/WideBar.h b/launcher/ui/widgets/WideBar.h
index 8ff62ef2..a0a7896c 100644
--- a/launcher/ui/widgets/WideBar.h
+++ b/launcher/ui/widgets/WideBar.h
@@ -2,9 +2,10 @@
#include <QAction>
#include <QMap>
+#include <QMenu>
#include <QToolBar>
-class QMenu;
+#include <memory>
class WideBar : public QToolBar {
Q_OBJECT
@@ -12,7 +13,7 @@ class WideBar : public QToolBar {
public:
explicit WideBar(const QString& title, QWidget* parent = nullptr);
explicit WideBar(QWidget* parent = nullptr);
- virtual ~WideBar();
+ ~WideBar() override = default;
void addAction(QAction* action);
void addSeparator();
@@ -23,12 +24,31 @@ class WideBar : public QToolBar {
void insertActionAfter(QAction* after, QAction* action);
QMenu* createContextMenu(QWidget* parent = nullptr, const QString& title = QString());
+ void showVisibilityMenu(const QPoint&);
+
+ // Ideally we would use a QBitArray for this, but it doesn't support string conversion,
+ // so using it in settings is very messy.
+
+ [[nodiscard]] QByteArray getVisibilityState() const;
+ void setVisibilityState(QByteArray&&);
private:
- struct BarEntry;
+ struct BarEntry {
+ enum class Type { None, Action, Separator, Spacer } type = Type::None;
+ QAction* bar_action = nullptr;
+ QAction* menu_action = nullptr;
+ };
- auto getMatching(QAction* act) -> QList<BarEntry*>::iterator;
+ auto getMatching(QAction* act) -> QList<BarEntry>::iterator;
+
+ /** Used to distinguish between versions of the WideBar with different actions */
+ [[nodiscard]] QByteArray getHash() const;
+ [[nodiscard]] bool checkHash(QByteArray const&) const;
private:
- QList<BarEntry*> m_entries;
+ QList<BarEntry> m_entries;
+
+ // Menu to toggle visibility from buttons in the bar
+ std::unique_ptr<QMenu> m_bar_menu = nullptr;
+ enum class MenuState { Fresh, Dirty } m_menu_state = MenuState::Dirty;
};