Skip to content

Damage overlay now shows building health in tooltip - #1811

Merged
Sephirex95 merged 14 commits into
Keriew:masterfrom
eugenikus8:tooltip-damage
Aug 1, 2026
Merged

Damage overlay now shows building health in tooltip#1811
Sephirex95 merged 14 commits into
Keriew:masterfrom
eugenikus8:tooltip-damage

Conversation

@eugenikus8

@eugenikus8 eugenikus8 commented Jun 9, 2026

Copy link
Copy Markdown

- Hide fire, damage, and crime tooltips on non-building tiles
- Display roadblock on all overlay layers

- Damage overlay now shows building health in tooltip

Details hp bar (enemy overlay) 2026-06-11_135733

- Display actual WHARF efficiency in tooltip

Details Display actual WHARF efficiency in tooltip
2026-06-23_121047

- Fix black tiles when undo bridges

Details bug
2026-06-23_140244

@eugenikus8
eugenikus8 marked this pull request as draft June 9, 2026 18:35
@eugenikus8
eugenikus8 marked this pull request as ready for review June 13, 2026 14:46
@eugenikus8

Copy link
Copy Markdown
Author

@Sephirex95 If you have some time, could you take a look at this PR?

@Sephirex95 Sephirex95 self-assigned this Jul 31, 2026

@Sephirex95 Sephirex95 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll just playtest this and should be OK to merge. Just have to check how these render and if they aren't too distracting

Comment thread src/building/destruction.c
Comment thread src/widget/city/overlay/risks.c Outdated
Comment thread src/widget/city/overlay/risks.c Outdated
@Sephirex95

Sephirex95 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Correct these aspects and request a re-review please:

In Enemies overlay, the bars are shown, but the tooltip is missing. Since Enemies overlay is primarily used for fights, and damage for checking the engineering status, I think that tooltip is more relevant here.
image

In Damage overlay, walls, towers, palisades and watchtowers get flattened but the hp bar stays - for consistency I would remove the bars and leave just tooltip, if at all. Since engineers do not repair buildings, this information isn't relevant to the primary goal of the overlayimage

Yeah, after thinking about it for a moment, I would suggest moving both tooltip and bars to the Enemy overlay and leaving Damage untouched. While I understand that it creates an odd naming inconsistency, this is just a vanilla legacy naming convention. We could rename this overlay to Risks->Collapse tbh

@Sephirex95 Sephirex95 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please adjust the requested and ping me :)

@eugenikus8

Copy link
Copy Markdown
Author

@Sephirex95 done

@Sephirex95
Sephirex95 merged commit 01f774b into Keriew:master Aug 1, 2026
37 of 38 checks passed
@Sephirex95

Copy link
Copy Markdown
Collaborator

Well done! Merged :)

@eugenikus8
eugenikus8 deleted the tooltip-damage branch August 1, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants