Skip to content

Extend the Harvester counter color definition and standardize the variable names#2163

Merged
Coronia merged 2 commits intoPhobos-developers:developfrom
DeathFishAtEase:colorgreen
Apr 7, 2026
Merged

Extend the Harvester counter color definition and standardize the variable names#2163
Coronia merged 2 commits intoPhobos-developers:developfrom
DeathFishAtEase:colorgreen

Conversation

@DeathFishAtEase
Copy link
Copy Markdown
Collaborator

  1. Allow defining the color of the "green" state of Harvester counter independently of ToolTipColor.
  2. Align the variable names related to color definitions of the old Harvester counter and Power delta counter that do not correspond to INI flags with the structure of INI flags.

@DeathFishAtEase DeathFishAtEase added Needs testing ⚙️T1 T1 maintainer review is sufficient No Documentation Needed No documentation needed whatsoever labels Apr 6, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@phoboscn-bot
Copy link
Copy Markdown

To Chinese users:
This pull request has been mentioned on Phobos CN. There might be relevant details there:

致中文用户:
此拉取请求已在 Phobos CN 上被提及。那里可能有相关详细信息:

https://www.phoboscn.top/t/topic/339/1

@TaranDahl TaranDahl added No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions. and removed Needs testing labels Apr 7, 2026
@Coronia Coronia merged commit d254c26 into Phobos-developers:develop Apr 7, 2026
21 of 24 checks passed
@DeathFishAtEase DeathFishAtEase deleted the colorgreen branch April 8, 2026 12:03
DeathFishAtEase added a commit to DeathFishAtEase/Phobos that referenced this pull request Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Documentation Needed No documentation needed whatsoever No test needed This PR is simple enough, or changes no in-game logic, so no in-game testing is required. ⚙️T1 T1 maintainer review is sufficient Will be merged in 24h This PR will be merged in 24 hours if no one has further instructions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants