mirror of
https://github.com/openfrontio/OpenFrontIO.git
synced 2026-06-21 07:40:43 +00:00
Fix labels in public lobby modifier display 🏷️ (#3456)
## Description: Fixes two issues in the join lobby modal's modifier display: 1. **Starting Gold**: Label showed "Starting Gold (Millions)" with value "5M", duplicating "millions". Now shows "Starting Gold" as the label, keeping "5M" as the value. 2. **Disable Alliances**: Label showed "Disable Alliances" with value "Enabled", which is confusing. Now shows "Alliances" as the label with "Disabled" as the value. Join Lobby Modal needs a general rework, I will probably make an PR ## Please complete the following: - [X] I have added screenshots for all UI updates - [X] I process any text displayed to the user through translateText() and I've added it to the en.json file - [X] I have added relevant tests to the test directory - [X] I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced ## Please put your Discord username so you can be contacted if a bug or regression is found: FloPinguin
This commit is contained in:
@@ -24,6 +24,7 @@
|
||||
"copied": "Copied!",
|
||||
"click_to_copy": "Click to copy",
|
||||
"enabled": "Enabled",
|
||||
"disabled": "Disabled",
|
||||
"map_default": "Map default"
|
||||
},
|
||||
"main": {
|
||||
@@ -488,8 +489,10 @@
|
||||
"crowded": "Crowded",
|
||||
"hard_nations": "Hard Nations",
|
||||
"starting_gold": "{amount}M Starting Gold",
|
||||
"starting_gold_label": "Starting Gold",
|
||||
"gold_multiplier": "x{amount} Gold Multiplier",
|
||||
"disable_alliances": "Alliances Disabled"
|
||||
"disable_alliances": "Alliances Disabled",
|
||||
"disable_alliances_label": "Alliances"
|
||||
},
|
||||
"select_lang": {
|
||||
"title": "Select Language"
|
||||
|
||||
+3
-2
@@ -159,7 +159,7 @@ export function getActiveModifiers(
|
||||
(modifiers.startingGold / 1_000_000).toPrecision(12),
|
||||
);
|
||||
result.push({
|
||||
labelKey: "host_modal.starting_gold",
|
||||
labelKey: "public_game_modifier.starting_gold_label",
|
||||
badgeKey: "public_game_modifier.starting_gold",
|
||||
badgeParams: {
|
||||
amount: millions,
|
||||
@@ -181,8 +181,9 @@ export function getActiveModifiers(
|
||||
}
|
||||
if (modifiers.isAlliancesDisabled) {
|
||||
result.push({
|
||||
labelKey: "host_modal.disable_alliances",
|
||||
labelKey: "public_game_modifier.disable_alliances_label",
|
||||
badgeKey: "public_game_modifier.disable_alliances",
|
||||
formattedValue: translateText("common.disabled"),
|
||||
});
|
||||
}
|
||||
return result;
|
||||
|
||||
Reference in New Issue
Block a user