#3397 closed enhancement (fixed)
Option to select between generic and specific name or to select both
Reported by: | Stan | Owned by: | Freagarach |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 25 |
Component: | UI – Miscellaneous | Keywords: | |
Cc: | Patch: | Phab:D2126 |
Description (last modified by )
As stated in the discussion here :
http://wildfiregames.com/forum/index.php?s=541f7aea38c912e7f965f18c8f0cc04d&showtopic=20031
it would be nice to be able to switch between general and specific name, because it seems it confuse new players.
Feneur simply stated that instead of reading you could just pick the icons, however wowgetoffyourcellphone and OP thinks specific names should be off by default.
Attachments (3)
Change History (15)
comment:1 by , 9 years ago
Keywords: | simple added |
---|
comment:2 by , 9 years ago
I agree with you nonetheless I think a drop down could be nice. So everyone is happy ;)
comment:3 by , 9 years ago
Replying to stanislas69:
however wowgetoffyourcellphone and OP thinks specific names should be off by default.
I don't agree with it being default, but I have always been in favor of this option, and similarly a "simple/basic" UI option with less information, and an "advanced/hardcore" option with lots of stats and whatnot. But for this ticket, it should be easy to add an option as elexis says.
The issue with a dropdown IIRC is the config screen doesn't currently support that.
by , 8 years ago
Attachment: | t3397_v1.patch added |
---|
comment:4 by , 8 years ago
Keywords: | review patch added |
---|---|
Milestone: | Backlog → Alpha 20 |
Summary: | Option to select between generic and specific name or to select both → [PATCH] Option to select between generic and specific name or to select both |
by , 8 years ago
Attachment: | t3397_v2.patch added |
---|
by , 8 years ago
Attachment: | t3397_v3.patch added |
---|
comment:5 by , 8 years ago
Milestone: | Alpha 20 → Alpha 21 |
---|
comment:6 by , 8 years ago
Keywords: | review removed |
---|
Hi, really sorry for the huge delay in reviewing :(
- The option should not be a number but a boolean (either display specific names or don't)
- Please add a description of the option and fix the missing space
- I think the new
getTemplateLabels
function is not really wanted. It makes you rely on checks likeif (specific != generic)
which could cause problems. Better get the config value and take decisions based on it. You might have to hide GUI elements when the specific name is not wanted.
Thanks for your work so far!
comment:7 by , 8 years ago
Milestone: | Alpha 21 → Backlog |
---|---|
Type: | task → enhancement |
Backlogging due to lack of progress.
comment:8 by , 5 years ago
Component: | UI & Simulation → Misc. UI |
---|
Move tickets to Misc. UI
as "UI & Simulation" got some sub components.
comment:9 by , 4 years ago
Description: | modified (diff) |
---|---|
Milestone: | Backlog → Work In Progress |
Patch: | → Phab:D2126 |
comment:11 by , 3 years ago
Keywords: | simple patch removed |
---|---|
Milestone: | Work In Progress → Alpha 25 |
comment:12 by , 3 years ago
Summary: | [PATCH] Option to select between generic and specific name or to select both → Option to select between generic and specific name or to select both |
---|
I think noone will be so hardcore to only show the civ-specific names in the building / unit panel. Personally I don't see the need for a setting that hides the civ-specific names, as the generic names are always shown in parenthesis.
But if many new players would prefer that, maybe we could make a boolean setting that if enabled, only shows the generic names.
Wouldn't be hard to implement. See r16927 for an example of a new option.
In the template XML files, the names are given as:
If you search for
GenericName
andSpecificName
in *.js files, you will find all occurances that should be changed, if that option is active.