Skip to content

feat(navigation): Add Live TV Navbar toggle and restructure settings#450

Open
mattsigal wants to merge 1 commit into
Moonfin-Client:mainfrom
mattsigal:feature/add-livetv-nav-toggle
Open

feat(navigation): Add Live TV Navbar toggle and restructure settings#450
mattsigal wants to merge 1 commit into
Moonfin-Client:mainfrom
mattsigal:feature/add-livetv-nav-toggle

Conversation

@mattsigal

@mattsigal mattsigal commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Summary

Restructures the navigation settings layout and adds a conditional "Show Live TV" toolbar/navbar button.

Related Issues

Discord requested.

  • Closes #
  • Fixes #
  • Related to #

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • UI/UX update
  • Documentation update
  • Build/CI change
  • Other (describe):

Changes Made

  • Restructured the navigation settings panel and screen under "Navbar Display" and "Navbar Contents" headers.
  • Added descriptive guide text under "Navbar Contents" and removed individual toggle subtitles.
  • Added a "Show Live TV" preference toggle and integration for navigation shortcuts in the left sidebar, top toolbar, and mobile bottom nav using the live_tv icon.
  • Dynamically hides the "Show Live TV" preference option if Live TV is not configured on the server (checking for channel existence).
  • Hidden the redundant "Show in navigation" toggle under Live TV Library Visibility settings.
  • Standardized ALL REFERENCES in-app to use "Live TV" (using a space) instead of "LiveTV" for consistency.
  • Added safety checks to make sure Live TV icon is never shown if the service is not available

Platform

  • Android
  • iOS
  • macOS
  • Windows
  • Linux
  • All / Shared code

Testing

Describe how this change was tested.

  • Tested on emulator / simulator
  • Tested on physical device
  • Manual testing completed
  • Not tested (explain why):

Test Steps

  1. Compiled the local application using build-windows.ps1.
  2. Verified that compilation finished successfully.
  3. With Live TV enabled on the server, check for settings present.
  4. With Live TV disabled on the server, check for settings hidden.

Screenshots (if applicable)

New icon in navbar:

2026-06-09_07-55-25_moonfin

New toggle

2026-06-09_07-55-57_moonfin

Toggle Hidden in Refined UI

2026-06-09_08-12-47_moonfin 2026-06-09_08-12-38_moonfin

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

@mattsigal mattsigal changed the title feat(navigation): Add Live TV Navabar toggle and restructure settings feat(navigation): Add Live TV Navbar toggle and restructure settings Jun 9, 2026
@mattsigal mattsigal force-pushed the feature/add-livetv-nav-toggle branch from 07fe60c to 8420a74 Compare June 9, 2026 16:48
@mattsigal

Copy link
Copy Markdown
Contributor Author

New commit brought back the library toggle so someone can have it in their library view, while others can have it in their navbar.

…settings

- Restructured navigation settings screen and side panel under "Navbar Display" and "Navbar Contents" headers.
- Added descriptive guide text under "Navbar Contents" and removed individual toggle subtitles.
- Added "Show Live TV" preference and conditional navigation button (Icons.live_tv) in left sidebar, top toolbar, and mobile bottom nav.
- Hides the "Show Live TV" preference tile and hides the icon entirely if Live TV is not configured on the server.
- Automatically resets the "Show Live TV" preference to false if no Live TV channels are configured on the server.
- Removed redundant "Show in navigation" toggle for Live TV libraries under Library Visibility settings.
@mattsigal mattsigal force-pushed the feature/add-livetv-nav-toggle branch from 8420a74 to a571698 Compare June 11, 2026 04:12
@RadicalMuffinMan RadicalMuffinMan force-pushed the main branch 2 times, most recently from 148433d to 537b010 Compare June 19, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants