diff options
| -rw-r--r-- | data/ui/options_dialog.ui | 21 | ||||
| -rw-r--r-- | src/aoapplication.cpp | 16 | ||||
| -rw-r--r-- | src/charselect.cpp | 4 | ||||
| -rw-r--r-- | src/courtroom.cpp | 29 | ||||
| -rw-r--r-- | src/courtroom.h | 3 | ||||
| -rw-r--r-- | src/lobby.cpp | 15 | ||||
| -rw-r--r-- | src/lobby.h | 3 | ||||
| -rw-r--r-- | src/options.cpp | 25 | ||||
| -rw-r--r-- | src/options.h | 8 | ||||
| -rw-r--r-- | src/widgets/aooptionsdialog.cpp | 8 | ||||
| -rw-r--r-- | src/widgets/aooptionsdialog.h | 1 |
11 files changed, 110 insertions, 23 deletions
diff --git a/data/ui/options_dialog.ui b/data/ui/options_dialog.ui index 19100cde..bcdb7a44 100644 --- a/data/ui/options_dialog.ui +++ b/data/ui/options_dialog.ui @@ -39,9 +39,9 @@ <property name="geometry"> <rect> <x>0</x> - <y>-511</y> + <y>-555</y> <width>394</width> - <height>850</height> + <height>872</height> </rect> </property> <layout class="QFormLayout" name="formLayout"> @@ -573,6 +573,23 @@ </property> </widget> </item> + <item row="34" column="1"> + <widget class="QCheckBox" name="restoreposition_cb"> + <property name="text"> + <string/> + </property> + </widget> + </item> + <item row="34" column="0"> + <widget class="QLabel" name="restoreposition_lbl"> + <property name="toolTip"> + <string>If ticked, some windows restore their last known position where they were closed.</string> + </property> + <property name="text"> + <string>Restore Window Position</string> + </property> + </widget> + </item> </layout> </widget> </widget> diff --git a/src/aoapplication.cpp b/src/aoapplication.cpp index 0b73575b..9abc958c 100644 --- a/src/aoapplication.cpp +++ b/src/aoapplication.cpp @@ -51,10 +51,18 @@ void AOApplication::construct_lobby() w_lobby = new Lobby(this, net_manager); - QRect geometry = QGuiApplication::primaryScreen()->geometry(); - int x = (geometry.width() - w_lobby->width()) / 2; - int y = (geometry.height() - w_lobby->height()) / 2; - w_lobby->move(x, y); + auto point = Options::getInstance().windowPosition("lobby"); + if (!Options::getInstance().restoreWindowPositionEnabled() || !point.has_value()) + { + QRect geometry = QGuiApplication::primaryScreen()->geometry(); + int x = (geometry.width() - w_lobby->width()) / 2; + int y = (geometry.height() - w_lobby->height()) / 2; + w_lobby->move(x, y); + } + else + { + w_lobby->move(point->x(), point->y()); + } if (Options::getInstance().discordEnabled()) { diff --git a/src/charselect.cpp b/src/charselect.cpp index 8bce410a..d36220aa 100644 --- a/src/charselect.cpp +++ b/src/charselect.cpp @@ -230,9 +230,9 @@ void Courtroom::on_char_button_context_menu_requested(const QPoint &pos) } QMenu *menu = new QMenu(this); - menu->addAction(QString("Edit " + char_name + "/char.ini"), this, [=] { QDesktopServices::openUrl(QUrl::fromLocalFile(char_ini_path)); }); + menu->addAction(QString("Edit " + char_name + "/char.ini"), this, [=, this] { QDesktopServices::openUrl(QUrl::fromLocalFile(char_ini_path)); }); menu->addSeparator(); - menu->addAction(QString("Open character folder " + char_name), this, [=] { + menu->addAction(QString("Open character folder " + char_name), this, [=, this] { QString p_path = ao_app->get_real_path(VPath("characters/" + char_name + "/")); if (!dir_exists(p_path)) { diff --git a/src/courtroom.cpp b/src/courtroom.cpp index 7a439586..45fb7d61 100644 --- a/src/courtroom.cpp +++ b/src/courtroom.cpp @@ -8,13 +8,11 @@ Courtroom::Courtroom(AOApplication *p_ao_app) : QMainWindow() + , ao_app{p_ao_app} { - ao_app = p_ao_app; - - this->setWindowFlags((this->windowFlags() | Qt::CustomizeWindowHint) & ~Qt::WindowMaximizeButtonHint); + setWindowFlags((this->windowFlags() | Qt::CustomizeWindowHint) & ~Qt::WindowMaximizeButtonHint); ao_app->initBASS(); - keepalive_timer = new QTimer(this); keepalive_timer->start(45000); @@ -1372,6 +1370,16 @@ void Courtroom::done_received() show(); + if (Options::getInstance().restoreWindowPositionEnabled()) + { + auto maybe_point = Options::getInstance().windowPosition("courtroom"); + if (maybe_point.has_value()) + { + qDebug() << maybe_point.value(); + QMainWindow::move(maybe_point.value()); + } + } + ui_spectator->show(); } @@ -1907,6 +1915,13 @@ void Courtroom::set_judge_buttons() show_judge_controls(ao_app->get_pos_is_judge(current_or_default_side())); } +void Courtroom::closeEvent(QCloseEvent *event) +{ + Options::getInstance().setWindowPosition("courtroom", pos()); + qDebug() << pos(); + QMainWindow::closeEvent(event); +} + void Courtroom::on_chat_return_pressed() { if (is_muted) @@ -5191,7 +5206,7 @@ void Courtroom::on_pos_dropdown_context_menu_requested(const QPoint &pos) QMenu *menu = new QMenu(ui_iniswap_dropdown); menu->setAttribute(Qt::WA_DeleteOnClose); - menu->addAction(QString("Open background " + current_background), this, [=] { + menu->addAction(QString("Open background " + current_background), this, [=, this] { QString p_path = ao_app->get_real_path(VPath("background/" + current_background + "/")); if (!dir_exists(p_path)) { @@ -5303,7 +5318,7 @@ void Courtroom::on_iniswap_context_menu_requested(const QPoint &pos) } menu->addSeparator(); - menu->addAction(QString("Open character folder " + current_char), this, [=] { + menu->addAction(QString("Open character folder " + current_char), this, [=, this] { QString p_path = ao_app->get_real_path(VPath("characters/" + current_char + "/")); if (!dir_exists(p_path)) { @@ -6127,7 +6142,7 @@ void Courtroom::on_text_color_context_menu_requested(const QPoint &pos) QMenu *menu = new QMenu(this); menu->setAttribute(Qt::WA_DeleteOnClose); - menu->addAction(QString("Open currently used chat_config.ini"), this, [=] { + menu->addAction(QString("Open currently used chat_config.ini"), this, [=, this] { QString p_path = ao_app->get_asset("chat_config.ini", Options::getInstance().theme(), Options::getInstance().subTheme(), ao_app->default_theme, ao_app->get_chat(current_char)); if (!file_exists(p_path)) { diff --git a/src/courtroom.h b/src/courtroom.h index c0fff0ad..ba771acf 100644 --- a/src/courtroom.h +++ b/src/courtroom.h @@ -297,6 +297,9 @@ public: void set_judge_state(JudgeState new_state); void set_judge_buttons(); +protected: + virtual void closeEvent(QCloseEvent *event) override; + private: AOApplication *ao_app; diff --git a/src/lobby.cpp b/src/lobby.cpp index 5fb0f506..f4da9d54 100644 --- a/src/lobby.cpp +++ b/src/lobby.cpp @@ -11,11 +11,10 @@ #include <QUiLoader> Lobby::Lobby(AOApplication *p_ao_app, NetworkManager *p_net_manager) - : QMainWindow() + : QMainWindow{} + , ao_app{p_ao_app} + , net_manager{p_net_manager} { - ao_app = p_ao_app; - net_manager = p_net_manager; - reloadUi(); } @@ -82,6 +81,12 @@ int Lobby::pageSelected() return current_page; } +void Lobby::closeEvent(QCloseEvent *event) +{ + Options::getInstance().setWindowPosition("lobby", pos()); + QMainWindow::closeEvent(event); +} + void Lobby::reset_selection() { last_index = -1; @@ -162,7 +167,7 @@ void Lobby::loadUI() FROM_UI(QTextBrowser, server_description_text); FROM_UI(QPushButton, connect_button); connect(ui_connect_button, &QPushButton::released, net_manager, &NetworkManager::join_to_server); - connect(ui_connect_button, &QPushButton::released, this, [=] { ui_server_player_count_lbl->setText(tr("Joining Server...")); }); + connect(ui_connect_button, &QPushButton::released, this, [=, this] { ui_server_player_count_lbl->setText(tr("Joining Server...")); }); connect(net_manager, &NetworkManager::server_connected, ui_connect_button, &QPushButton::setEnabled); FROM_UI(QTextBrowser, motd_text); diff --git a/src/lobby.h b/src/lobby.h index 4b4d6240..8b894f4c 100644 --- a/src/lobby.h +++ b/src/lobby.h @@ -31,6 +31,9 @@ public: int get_selected_server(); int pageSelected(); +protected: + void closeEvent(QCloseEvent *event) override; + private: AOApplication *ao_app; NetworkManager *net_manager; diff --git a/src/options.cpp b/src/options.cpp index 9145aa4f..0b963d60 100644 --- a/src/options.cpp +++ b/src/options.cpp @@ -763,3 +763,28 @@ QString Options::getUIAsset(QString f_asset_name) qWarning() << "Unable to locate ui-asset" << f_asset_name << "in theme" << theme() << "Defaulting to embeeded asset."; return QString(":/data/ui/" + f_asset_name); } + +void Options::setWindowPosition(QString widget, QPoint position) +{ + config.setValue("windows/position_" + widget, position); +} + +std::optional<QPoint> Options::windowPosition(QString widget) +{ + QPoint point = config.value("windows/position_" + widget, QPoint()).toPoint(); + if (point.isNull()) + { + return std::nullopt; + } + return std::optional<QPoint>(point); +} + +bool Options::restoreWindowPositionEnabled() const +{ + return config.value("windows/restore", false).toBool(); +} + +void Options::setRestoreWindowPositionEnabled(bool state) +{ + config.setValue("windows/restore", state); +} diff --git a/src/options.h b/src/options.h index 4ccc3016..f28dfcd3 100644 --- a/src/options.h +++ b/src/options.h @@ -5,6 +5,8 @@ #include <QCoreApplication> #include <QSettings> +#include <QPoint> + class Options { public: @@ -269,6 +271,12 @@ public: // Theming Nonesense! QString getUIAsset(QString f_asset_name); + void setWindowPosition(QString widget, QPoint position); + std::optional<QPoint> windowPosition(QString widget); + + bool restoreWindowPositionEnabled() const; + void setRestoreWindowPositionEnabled(bool state); + private: /** * @brief QSettings object for config.ini diff --git a/src/widgets/aooptionsdialog.cpp b/src/widgets/aooptionsdialog.cpp index 4d55652e..db3c8788 100644 --- a/src/widgets/aooptionsdialog.cpp +++ b/src/widgets/aooptionsdialog.cpp @@ -157,10 +157,10 @@ void AOOptionsDialog::registerOption(const QString &widgetName, V (Options::*get } OptionEntry entry; - entry.load = [=] { + entry.load = [=, this] { setWidgetData<T, V>(widget, (Options::getInstance().*getter)()); }; - entry.save = [=] { + entry.save = [=, this] { (Options::getInstance().*setter)(widgetData<T, V>(widget)); }; @@ -323,7 +323,7 @@ void AOOptionsDialog::setupUI() connect(ui_theme_reload_button, &QPushButton::clicked, this, &::AOOptionsDialog::onReloadThemeClicked); FROM_UI(QPushButton, theme_folder_button); - connect(ui_theme_folder_button, &QPushButton::clicked, this, [=] { + connect(ui_theme_folder_button, &QPushButton::clicked, this, [=, this] { QString p_path = ao_app->get_real_path(ao_app->get_theme_path("", ui_theme_combobox->itemText(ui_theme_combobox->currentIndex()))); if (!dir_exists(p_path)) { @@ -359,6 +359,7 @@ void AOOptionsDialog::setupUI() FROM_UI(QCheckBox, sfx_on_idle_cb); FROM_UI(QCheckBox, evidence_double_click_cb); FROM_UI(QCheckBox, slides_cb); + FROM_UI(QCheckBox, restoreposition_cb); registerOption<QSpinBox, int>("theme_scaling_factor_sb", &Options::themeScalingFactor, &Options::setThemeScalingFactor); registerOption<QCheckBox, bool>("animated_theme_cb", &Options::animatedThemeEnabled, &Options::setAnimatedThemeEnabled); @@ -402,6 +403,7 @@ void AOOptionsDialog::setupUI() registerOption<QCheckBox, bool>("sfx_on_idle_cb", &Options::playSelectedSFXOnIdle, &Options::setPlaySelectedSFXOnIdle); registerOption<QCheckBox, bool>("evidence_double_click_cb", &Options::evidenceDoubleClickEdit, &Options::setEvidenceDoubleClickEdit); registerOption<QCheckBox, bool>("slides_cb", &Options::slidesEnabled, &Options::setSlidesEnabled); + registerOption<QCheckBox, bool>("restoreposition_cb", &Options::restoreWindowPositionEnabled, &Options::setRestoreWindowPositionEnabled); // Callwords tab. This could just be a QLineEdit, but no, we decided to allow // people to put a billion entries in. diff --git a/src/widgets/aooptionsdialog.h b/src/widgets/aooptionsdialog.h index b16be331..eb64a539 100644 --- a/src/widgets/aooptionsdialog.h +++ b/src/widgets/aooptionsdialog.h @@ -48,6 +48,7 @@ private: QPushButton *ui_theme_folder_button; QCheckBox *ui_evidence_double_click_cb; QCheckBox *ui_slides_cb; + QCheckBox *ui_restoreposition_cb; QCheckBox *ui_animated_theme_cb; QSpinBox *ui_stay_time_spinbox; QCheckBox *ui_instant_objection_cb; |
