Page MenuHomeWildfire Games

x → × in game speeds
ClosedPublic

Authored by Nescio on Jul 20 2020, 11:37 AM.

Details

Reviewers
Imarok
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP23871: x → × in game speeds
Summary

This patch properly formats game speed trings by replacing the x (X) with a × (multiplication symbol), per the style guide and because it looks better in game.
See also D2610.

Test Plan

Tiny and trivial to test.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Nescio created this revision.Jul 20 2020, 11:37 AM
Owners added a subscriber: Restricted Owners Package.Jul 20 2020, 11:37 AM
Stan added a comment.Jul 20 2020, 11:38 AM

Screenshot?

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

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2716/display/redirect

Imarok added a subscriber: Imarok.Jul 22 2020, 5:21 PM

Sounds reasonable.

Quick grep for completion shows:

  • simulation/data/civs/cart.json: Carthaginian walls, gates, and towers have 3x health of a standard wall, but also 2x build time.
  • simulation/templates/template_unit_cavalry_melee_spearman.xml, simulation/templates/template_unit_hero_cavalry_spearman.xml, simulation/templates/template_unit_hero_infantry_spearman.xml, simulation/templates/template_unit_champion_cavalry_spearman.xml, simulation/templates/template_unit_hero_infantry_pikeman.xml, simulation/templates/template_unit_champion_infantry_pikeman.xml, simulation/templates/template_unit_infantry_melee_spearman.xml, simulation/templates/template_unit_champion_infantry_spearman.xml, simulation/templates/template_unit_infantry_melee_pikeman.xml: 1.5x vs. Cavalry. or similar.

iirc there was a diff for the counters. Not sure if it included the x× change.

iirc there was a diff for the counters.

D2610

iirc there was a diff for the counters.

D2610

ok, but the cart wall tooltip is still wrong, right?

simulation/data/civs/cart.json:

Included in D2885.

Also, you might like D2820 too. (It's really tiny.)

Imarok accepted this revision.Jul 22 2020, 6:10 PM

simulation/data/civs/cart.json:

Included in D2885.

Ok

This revision is now accepted and ready to land.Jul 22 2020, 6:10 PM
This revision was automatically updated to reflect the committed changes.