Opened 16 months ago

Closed 16 months ago

#6694 closed defect (fixed)

"Requires Town Phase" and "Requires City Phase" should be in red

Reported by: abian Owned by: abian
Priority: Should Have Milestone: Alpha 27
Component: UI & Simulation Keywords: simple
Cc: abian Patch: ​Phab:D4899

Description

The text of the lines "Requires Town Phase" and "Requires City Phase" should be in red when warning of unfulfilled requirements, just like the "Insufficient resources" message and others.

Attachments (2)

requires-phase.png​ (224.7 KB ) - added by abian 16 months ago.
other-requirements-in-red.png​ (193.1 KB ) - added by abian 16 months ago.

Download all attachments as: .zip

Change History (7)

by abian, 16 months ago

Attachment: requires-phase.png​ added

by abian, 16 months ago

Attachment: other-requirements-in-red.png​ added

comment:1 by Freagarach, 16 months ago

Keywords: simple added
Milestone: → Backlog

Sounds like a good thing. Also not too hard, maybe a good beginner's ticket. :)

comment:2 by abian, 16 months ago

Owner: set to abian
Patch: → D4899
Status: new → assigned

comment:3 by abian, 16 months ago

Patch: D4899 → Phab:D4899

comment:4 by Langbart, 16 months ago

Milestone: Backlog → Alpha 27

The patch has already been approved, move the milestone to A27 to not forget to commit it before release.

comment:5 by Freagarach, 16 months ago

Resolution: → fixed
Status: assigned → closed

In 27484:

Fix required technology tooltip colour.

Done by creating a function for 'objections' and using that for any unmet requirements.

Patch by: @abian
Differential revision: ​https://code.wildfiregames.com/D4899
Comments by: @Langbart
Fixes #6694

Note: See TracTickets for help on using tickets.