New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[D2K] Show unlocked things in tooltips #12069
Comments
Well, it looks fine but 1-) You miswrote Atreides. There is no s after A. |
1-) Removed the awkward s 2-) Concrete: Large concrete inherits from ^concrete and doesn't have its own tooltip, so ingame both concrete slabs are called "Concrete" LinkToLine Thumper: There is no 3-) Done So can I PR it? |
Hmm, i didn't check the OpenRA's unit names. Thumper is called Thumper Infantry in otiginal game. Looks like there are other inconsistent with original names. I think it would be good it you fix them too when you are in it.
Some other units/ buildings has extra words in it (Raider Trike, Missile Quad, Spice Rafinery etc.) when compared to original but i guess it is fine so don't touch them. I think you can file a PR after that. |
These tooltips are unreasonably wide, so you should add linebreaks. |
@MustaphaTR: Yeah the naming is generally a bit inconsistent (IX Research Centre (UK form: manual and in OpenRA yaml) <> IX Research Center (US form: gruntmodversion and online-wiki) @pchote: less wide version |
Default language should be consistently US English. |
Okay, I can do these renames but I don't know if there are any additional changes needed. The repository search listed me these files: Link
|
There shouldn't be any additional changes needed, except maps that may have IX Research Center placed, which is a thing map makers need to do, not you. None of the maps comes with OpenRA D2K mod have such as only Atr1-3 is made yet. But there is another thing, i can't check right now as my this computer doesn't run TibEd, but if original codename was research_centre, i think we should keep it. If this is true just change the visible name (i.e. Tooltip:) and not codename. |
Okay, I will only change the Tooltip |
Closed via #12547. |
The upgrade tooltips are too simple. It should show unlocked things.
The text was updated successfully, but these errors were encountered: