Skip to content

Commit 8e6a5bb

Browse files
committed
Renamed return and close methods and signals in GameOptionsDialog for clarity and consistency
1 parent 64c6198 commit 8e6a5bb

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

appOPHD/UI/GameOptionsDialog.cpp

+7-7
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@ GameOptionsDialog::GameOptionsDialog() :
1010
btnSave{"Save current game", {this, &GameOptionsDialog::onSave}},
1111
btnLoad{"Load a saved game", {this, &GameOptionsDialog::onLoad}},
1212
btnHelp{"Help", {this, &GameOptionsDialog::onHelp}},
13-
btnReturn{"Return to current game", {this, &GameOptionsDialog::onReturn}},
14-
btnClose{"Return to Main Menu", {this, &GameOptionsDialog::onClose}}
13+
btnReturn{"Return to current game", {this, &GameOptionsDialog::onReturnToGame}},
14+
btnReturnToMainMenu{"Return to Main Menu", {this, &GameOptionsDialog::onReturnToMainMenu}}
1515
{
1616
position({0, 0});
1717

18-
const auto buttons = std::array{&btnSave, &btnLoad, &btnHelp, &btnReturn, &btnClose};
18+
const auto buttons = std::array{&btnSave, &btnLoad, &btnHelp, &btnReturn, &btnReturnToMainMenu};
1919
auto position = mRect.position + NAS2D::Vector{buttonHorizontalMargin, buttonHeight};
2020
for (auto button : buttons)
2121
{
@@ -36,7 +36,7 @@ void GameOptionsDialog::onEnableChange()
3636
btnLoad.enabled(enabled());
3737
btnHelp.enabled(enabled());
3838
btnReturn.enabled(enabled());
39-
btnClose.enabled(enabled());
39+
btnReturnToMainMenu.enabled(enabled());
4040
}
4141

4242

@@ -62,12 +62,12 @@ void GameOptionsDialog::onHelp()
6262
shellOpenPath("https://wiki.outpost2.net/doku.php?id=outposthd:how_to_play");
6363
}
6464

65-
void GameOptionsDialog::onReturn()
65+
void GameOptionsDialog::onReturnToGame()
6666
{
6767
hide();
6868
}
6969

70-
void GameOptionsDialog::onClose()
70+
void GameOptionsDialog::onReturnToMainMenu()
7171
{
72-
mSignalClose();
72+
mSignalReturnToMainMenu();
7373
}

appOPHD/UI/GameOptionsDialog.h

+5-5
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ class GameOptionsDialog : public Window
1515

1616
ClickSignal::Source& saveGameDialogButtonSignal() { return mSignalSaveDialog; }
1717
ClickSignal::Source& loadGameDialogButtonSignal() { return mSignalLoadDialog; }
18-
ClickSignal::Source& returnToMainMenuButtonSignal() { return mSignalClose; }
18+
ClickSignal::Source& returnToMainMenuButtonSignal() { return mSignalReturnToMainMenu; }
1919

2020
private:
2121
const int buttonHeight = 25;
@@ -26,18 +26,18 @@ class GameOptionsDialog : public Window
2626
void onLoad();
2727
void onSave();
2828
void onHelp();
29-
void onReturn();
30-
void onClose();
29+
void onReturnToGame();
30+
void onReturnToMainMenu();
3131

3232
void onEnableChange() override;
3333

3434
Button btnSave;
3535
Button btnLoad;
3636
Button btnHelp;
3737
Button btnReturn;
38-
Button btnClose;
38+
Button btnReturnToMainMenu;
3939

4040
ClickSignal mSignalSaveDialog;
4141
ClickSignal mSignalLoadDialog;
42-
ClickSignal mSignalClose;
42+
ClickSignal mSignalReturnToMainMenu;
4343
};

0 commit comments

Comments
 (0)