Page MenuHomeWildfire Games

standardize game option names, minor changes
Needs ReviewPublic

Authored by abian on Sep 12 2023, 7:44 PM.

Details

Reviewers
Freagarach
sera
Summary
  • remove redundant/inconsistent "Enable" in the names of some boolean options
  • remove inconsistent periods in some option names
  • a couple of minor clarifications for "specific"/"generic" names and "TLS encryption"
Test Plan

Tested in the game.

Event Timeline

abian created this revision.Sep 12 2023, 7:44 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7282/display/redirect

abian published this revision for review.Sep 12 2023, 7:46 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8371/display/redirect

sera accepted this revision.Sep 12 2023, 10:14 PM
sera added a subscriber: sera.
sera added inline comments.
binaries/data/mods/public/gui/options/options.json
63–64

Before or after, neither sounds great but alas I can't think of a good one either ...

This revision is now accepted and ready to land.Sep 12 2023, 10:14 PM
Freagarach requested changes to this revision.Sep 13 2023, 7:33 AM
Freagarach added inline comments.
binaries/data/mods/public/gui/options/options.json
70
This revision now requires changes to proceed.Sep 13 2023, 7:33 AM
abian updated this revision to Diff 22256.Sep 13 2023, 5:50 PM

@sera: Yeah. :/ I hope the examples are enough. Thanks!
@Freagarach: Thanks too! I've changed the quotation marks, they look good in the game and I don't have to escape them. :D

I also want to align the circles of the boolean options to the left, as in other windows, so they're closer to the text and it's easier to see which circle corresponds to which option. But it will be in another patch, I have to locate the file. If I add you as reviewers, sorry in advance for the noise.

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7286/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8375/display/redirect