Skip to content

Implement always on default viewing mode#9341

Open
PikachuEXE wants to merge 3 commits into
FreeTubeApp:developmentfrom
PikachuEXE:feature/always-on-viewing-modes
Open

Implement always on default viewing mode#9341
PikachuEXE wants to merge 3 commits into
FreeTubeApp:developmentfrom
PikachuEXE:feature/always-on-viewing-modes

Conversation

@PikachuEXE

Copy link
Copy Markdown
Member

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

Closes #9326

Description

Add 2 more "default" viewing mode (I guess it's not "default" anymore but I don't know how to name it...

  • Full Window (Always On)
  • Full Screen (Always On)

Screenshots

image image

Testing

A. Existing modes

  • Set default mode to each of them
  • Play a video (in a playlist if you can coz it's easier to testing)
  • Switch to another mode
  • Play another video without leaving the watch page
  • Confirm whatever selected not applied

B. New modes

  • Set default mode to each of them
  • Play a video (in a playlist if you can coz it's easier to testing)
  • Switch to another mode
  • Play another video without leaving the watch page
  • Confirm whatever selected IS applied

Desktop

  • OS:
  • OS Version:
  • FreeTube version:

Additional context

Text to be discussed/updated~

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) June 29, 2026 06:10
@github-actions github-actions Bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 29, 2026
Comment thread static/locales/en-US.yaml Outdated
External Player: External Player ({externalPlayerName})
Full Window (Always On): Full Window (Always On)
Full Screen (Always On): Full Screen (Always On)
Tooltip: Most default viewing modes are only applied when switching to watch page (e.g. playing another video in a playlist won't be applied selected view mode again). Those modes with '(Always On)' are applied every time.

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc Jun 29, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe something like this?

Suggested change
Tooltip: Most default viewing modes are only applied when switching to watch page (e.g. playing another video in a playlist won't be applied selected view mode again). Those modes with '(Always On)' are applied every time.
Tooltip: Most default viewing modes are only applied at the start of each watch session. Changes made during the session persist until it ends. Always-on viewing modes remain active throughout the watch session.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Most default viewing modes are only applied at the start of each watch session. Changes made during the session persist until it ends. part looks ok
Always-on viewing modes remain active throughout the watch session. not so much coz "remain active" read strange (like you can't turn it off)
I think the text should explain that those modes makes full window/screen applied when switched to a new video regardless of whatever mode change made when viewing previous video and not sure how to make the text short :)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about this
Always-on viewing modes are applied whenever you switch to a new video, regardless of any viewing mode changes made during the previous video

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated take a look

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! What about the first part change i suggested, want to keep it like that or alter it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Forgot to change that too, updated

@PikachuEXE

Copy link
Copy Markdown
Member Author

@ChunkyProgrammer ~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: waiting for review For PRs that are complete, tested, and ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Default Viewing Mode: Fullscreen doesnt work with pasted Links after the first one

3 participants