clean up closeRequest usages

This commit is contained in:
RickyRister 2025-08-01 05:40:23 -07:00
parent 704db020de
commit d5f5e0595a
5 changed files with 5 additions and 11 deletions

View file

@ -1253,7 +1253,7 @@ void TabGame::createMenuItems()
aConcede = new QAction(this);
connect(aConcede, &QAction::triggered, this, &TabGame::actConcede);
aLeaveGame = new QAction(this);
connect(aLeaveGame, &QAction::triggered, this, [this] { closeRequest(); });
connect(aLeaveGame, &QAction::triggered, this, &TabGame::closeRequest);
aFocusChat = new QAction(this);
connect(aFocusChat, &QAction::triggered, sayEdit, qOverload<>(&LineEditCompleter::setFocus));
aCloseReplay = nullptr;
@ -1303,7 +1303,7 @@ void TabGame::createReplayMenuItems()
aFocusChat = nullptr;
aLeaveGame = nullptr;
aCloseReplay = new QAction(this);
connect(aCloseReplay, &QAction::triggered, this, [this] { closeRequest(); });
connect(aCloseReplay, &QAction::triggered, this, &TabGame::closeRequest);
phasesMenu = nullptr;
gameMenu = new QMenu(this);

View file

@ -39,7 +39,7 @@ TabMessage::TabMessage(TabSupervisor *_tabSupervisor,
vbox->addWidget(sayEdit);
aLeave = new QAction(this);
connect(aLeave, &QAction::triggered, this, [this] { closeRequest(); });
connect(aLeave, &QAction::triggered, this, &TabMessage::closeRequest);
messageMenu = new QMenu(this);
messageMenu->addAction(aLeave);
@ -86,11 +86,6 @@ QString TabMessage::getTabText() const
return tr("%1 - Private chat").arg(QString::fromStdString(otherUserInfo->name()));
}
bool TabMessage::closeRequest()
{
return close();
}
void TabMessage::closeEvent(QCloseEvent *event)
{
emit talkClosing(this);

View file

@ -47,7 +47,6 @@ public:
const ServerInfo_User &_otherUserInfo);
~TabMessage() override;
void retranslateUi() override;
bool closeRequest() override;
void tabActivated() override;
QString getUserName() const;
QString getTabText() const override;

View file

@ -103,7 +103,7 @@ TabRoom::TabRoom(TabSupervisor *_tabSupervisor,
hbox->addWidget(userList, 1);
aLeaveRoom = new QAction(this);
connect(aLeaveRoom, &QAction::triggered, this, [this] { closeRequest(); });
connect(aLeaveRoom, &QAction::triggered, this, &TabRoom::closeRequest);
roomMenu = new QMenu(this);
roomMenu->addAction(aLeaveRoom);

View file

@ -364,7 +364,7 @@ void TabSupervisor::addCloseButtonToTab(Tab *tab, int tabIndex, QAction *manager
// If managed, all close requests should go through the menu action
connect(closeButton, &CloseButton::clicked, this, [manager] { checkAndTrigger(manager, false); });
} else {
connect(closeButton, &CloseButton::clicked, tab, [tab] { tab->closeRequest(); });
connect(closeButton, &CloseButton::clicked, tab, &Tab::closeRequest);
}
tabBar()->setTabButton(tabIndex, closeSide, closeButton);
}