Page MenuHomeWildfire Games

Options button in game setup gui named "Settings"
Needs RevisionPublic

Authored by ffffffff on Dec 16 2017, 11:20 PM.

Details

Reviewers
bb
Summary

This can help open and setting game/setup options in game setup gui also when combined with possible change in options categorie order D797 and possible D816 cause else its not possible to open and change game settings while in gamesetup, which can be needed if you want to change something while hosting a new game ańd waiting for players.

button

page

maybe also in combine with bringing more options to gamesetup options page (from D797 idea)

Test Plan

test

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

ffffffff created this revision.Dec 16 2017, 11:20 PM
ffffffff edited the summary of this revision. (Show Details)Dec 21 2017, 3:38 PM
ffffffff added subscribers: mapkoc, Hannibal_Barca, causative.
ffffffff added a subscriber: elexis.

Tested, works. Very useful: new people always asking how to set batch size and maybe other options. Needed here and in lobby (D1055). D1055 conflicts with this patch if D1055 applied first.

Should trigger the same action as pressing the options dialog hotkey. The hotkey can likely be overwritten.
Labeling the button "Game Setup" is wrong if the page is already called "Game Setup", it should just be "Options" in accordance with the window title and other button(s) leading to that dialog.

Since the other patches propose to add more buttons from the main menu to individual GUI pages, we should determine a layout that fits for all buttons before committing the first button.

binaries/data/mods/public/gui/options/options.js
150

unneeded parentheses

The term "Options" is very near to "More Options", that we already have in the gui and that therefore can be easily mistaken. Therefore I thought about "Game Setup" respective from the options categorie (as it sets up the game in term). Maybe "Settings" term?

ffffffff updated this revision to Diff 4884.Dec 22 2017, 6:10 PM

Settings term

ffffffff edited the summary of this revision. (Show Details)Dec 22 2017, 6:11 PM
ffffffff retitled this revision from Game Setup/Options button in game setup gui to Options button in game setup gui named "Settings".Dec 22 2017, 11:54 PM

Isn't it better to rename More options to More settings and keep Options name consistent across game?

Yea. Good point. Just wanted to keep gamesetup gui and instead taking this name. I am feeling fine with "Settings". Other opinions?

The "More Options" button can become "More Settings" if we have to (after all its called Match Setup and gamesetup), but bb wants to kill that button anyhow.

Until then I fully agree with mapkoc in that we options dialog must be opened with a button called options.

Notice that it removes space from the tooltip and cheat hint string which is quite ungood considering that the length of the strings currently about fits for most civ tooltips.

somehow i cannot get the tooltip to the correct size as the function rightAlignCancelButton() for some reason not making it correct setted even if i pass instead of the "cancelGame" button name the "gameSetupButton" name.

so tooltip text underlayed the cheats enabled tooltip

bb added a comment.Jan 30 2018, 3:46 PM

More options button killed and related code renamed to settings, wouldn't this button be better positioned under/above the lobby button with an icon (perhaps the ai setting icon is usable as placeholder for now)

binaries/data/mods/public/gui/gamesetup/gamesetup.js
897

no need for newline

binaries/data/mods/public/gui/options/options.js
149–150

imo the button should just open the upper most tab (or if there is some kind of saving, the lastly viewed)

jes good idea. then we save the tooltip space.

binaries/data/mods/public/gui/gamesetup/gamesetup.js
897

ok

binaries/data/mods/public/gui/options/options.js
149–150

jes that would be the other idea or both having last tab opened saved per gui. so in game would that also nice.

bb requested changes to this revision.Feb 7 2018, 8:22 PM

as above

This revision now requires changes to proceed.Feb 7 2018, 8:22 PM