Page MenuHomeWildfire Games

Improve strings from rP19652

Authored by Itms on Jun 4 2017, 10:32 PM.



Addresses @scythetwirler's concerns about rP19652.

Test Plan


Diff Detail

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

Event Timeline

Itms created this revision.Jun 4 2017, 10:32 PM
elexis edited edge metadata.Jun 4 2017, 10:40 PM

In D583 causative mentioned that we shouldn't add half sentences. Should this one become a complete sentence too?

Itms added a comment.Jun 4 2017, 10:44 PM

I don't know, "Disabled" is not a sentence either. We could go with "Enabled, only if the map comes with treasures of course", but that makes the user sound stupid :)

The only wrong part of the previous string I can identitfy is that the map doesn't place it (it's the map script or map author that placed the treasure).
The new string works for me, but I don't know if it fixes the specific thing is that scythetwirler found awkward.

Vulcan added a subscriber: Vulcan.Jun 4 2017, 11:23 PM
Executing section Default...
Executing section Source...
Executing section JS...

| 396| »   »   "label":·"[color=\""·+·g_DescriptionHighlight·+·"\"]"·+·title.label·+·":"·+·"[/color]",
|    | [NORMAL] ESLintBear (no-useless-concat):
|    | Unexpected string concatenation of literals.

|  89| »   »   if·(playerData·==·null·||·playerData.Civ·&&·playerData.Civ·==·"gaia")
|    | [NORMAL] JSHintBear:
|    | Use '===' to compare with 'null'.

|  94| »   »   let·isAI·=·playerData.AI·&&·playerData.AI·!=·"";
|    | [NORMAL] JSHintBear:
|    | Use '!==' to compare with ''.

| 311| »   »   »   »   g_GameAttributes.settings.Ceasefire·==·0·?
|    | [NORMAL] JSHintBear:
|    | Use '===' to compare with '0'.
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/131/ for more details.

This revision was automatically updated to reflect the committed changes.
Vulcan added a comment.Jun 5 2017, 3:44 AM

Build has FAILED

Link to build: http://jw:8080/job/phabricator/1481/
See console output for more information: http://jw:8080/job/phabricator/1481/console