Skip to content

Fix stream stats toggle not hiding diagnostics overlays - #7

Merged
nafields merged 1 commit into
nafields:mainfrom
ck13ptcg:main
Jun 4, 2026
Merged

Fix stream stats toggle not hiding diagnostics overlays#7
nafields merged 1 commit into
nafields:mainfrom
ck13ptcg:main

Conversation

@ck13ptcg

Copy link
Copy Markdown
Contributor

Fixes an issue where the Show Stream Stats setting did not fully disable stream diagnostics overlays.

Changes:

  • Makes Show Stream Stats the visibility control for the compact stats HUD.
  • Hides the Connected lifecycle badge when Show Stream Stats is disabled.
  • Keeps existing behavior when Show Stream Stats is enabled.

Tested on Apple TV:

  • Show Stream Stats Off: no stats HUD, no Connected badge.
  • Show Stream Stats On: stats HUD and Connected badge visible.

Note:
While testing, I found that pressing L + R enables the diagnostics overlay. That shortcut may be worth documenting in the README or Stream Settings UI.

@ck13ptcg
ck13ptcg requested a review from nafields as a code owner May 31, 2026 02:16
@sibwaze

sibwaze commented May 31, 2026

Copy link
Copy Markdown

Hello @ck13ptcg
Thank you for the pull request. I've been waiting for this fix (#5) since April, 27.
Seems like developer is not publishing his current progress on the project on GitHub. Not sure why, this is a great product and also free-to-use
Just for info: I've successfully built it with your PR and the issue is resolved. Can you also see my feature request #6?
PS: I’ve done it, thanks to grok. Will send pull request today or tomorrow

@nafields

nafields commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Sorry new baby is home. Life is busy approving now. Happy to add you both to the project to manage things while I'm away/out. I didn't realize people were actually using this as initial feedback was lacking.

@nafields
nafields merged commit 59d8041 into nafields:main Jun 4, 2026
7 of 9 checks passed
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.

3 participants