Skip to content

feat(navigator): show player limits in the blue titan palette - #423

Closed
TheMeinerLP wants to merge 2 commits into
mainfrom
feat/navigator-limits-palette
Closed

TheMeinerLP wants to merge 2 commits into
mainfrom
feat/navigator-limits-palette

Conversation

@TheMeinerLP

Copy link
Copy Markdown
Contributor

Stacked on #419 (fix/navigator-task-names); merge that first, or retarget this PR once #419 lands.

Changes

  • Destination lore shows current/max (e.g. 12/100) instead of N online. When the provider reports no max (0) only the current count is shown; an unknown count still adds no line.
  • TaskPlayers returns the task's PlayerCount (online and max, summed over running services) instead of a bare int. PlayerCount already carried max, so no core/common/bridge type changed.
  • Navigator palette in the Titan blue/white family: inventory title, spawn entry, hotbar feather and each destination name are gradients, varied per destination. The count uses the Titan gradient, the slash is gray.
  • Click handling is unchanged.

Tests

  • DestinationPlayerLoreTest: current/max, current only (max 0), zero, gray slash, unknown adds no line.
  • TaskPlayersTest, NavigatorPlayerCountTest: switched to the PlayerCount API and N/100 lore.
  • NavigatorDestinationTest: Slender letter colors now #00b4d8 to #caf0f8.
  • NavigatorModuleTest: spawn name compared as plain text, since its color changed.

https://claude.ai/code/session_01GXose9rakuRNZoM3TP3xj1

@TheMeinerLP
TheMeinerLP requested a review from a team as a code owner October 13, 2026 10:35
@github-actions

Copy link
Copy Markdown
Contributor

Test results

  894 files    894 suites   41m 17s ⏱️
2 403 tests 2 401 ✅  2 💤 0 ❌
7 237 runs  7 154 ✅ 83 💤 0 ❌

Results for commit e20e57e.

@TheMeinerLP

Copy link
Copy Markdown
Contributor Author

Superseded by #450: the navigator now shows online/max and uses per-mode gradients chosen in the navigator refresh. A palette-only change can be reopened on top of main if wanted.

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.

1 participant