Skip to content

Add playback loop default option#7014

Closed
aziztouhami wants to merge 0 commit intoFreeTubeApp:developmentfrom
aziztouhami:development
Closed

Add playback loop default option#7014
aziztouhami wants to merge 0 commit intoFreeTubeApp:developmentfrom
aziztouhami:development

Conversation

@aziztouhami
Copy link
Copy Markdown
Contributor

@aziztouhami aziztouhami commented Mar 17, 2025

Add playback loop default option

Pull Request Type

  • Feature Implementation
  • Bugfix
  • Documentation
  • Other

Related Issue

Description

This pull request adds a default playback loop option to FreeTube. With this feature, users can enable looping for videos automatically, improving the playback experience for those who frequently use the loop functionality.

Changes include:

  • Adding a new setting for enabling/disabling the default loop option.
  • Modifying the player settings component to support this feature.
  • Ensuring the setting persists between sessions.

Screenshots

A screen recording has been provided instead of screenshots.

Screen.Record.for.the.new.feature.mp4

Testing

The code is straightforward, and no additional testing was required.

Desktop

OS: Windows 10 Professionnel
OS Version: 22H2
FreeTube Version: 0.23.2

Additional Context

This feature enhances the user's ability to control playback behavior, making looping more accessible without requiring manual toggling for each video.

@github-actions github-actions bot added PR: dependencies Pull requests that update a dependency file PR: waiting for review For PRs that are complete, tested, and ready for review labels Mar 17, 2025
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 17, 2025 00:52
@absidue
Copy link
Copy Markdown
Member

absidue commented Mar 17, 2025

Your branch should be based on the development branch not master, please remove the unnecessary branch merging so that this pull request only contains your own commit.

@absidue absidue added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Mar 17, 2025
auto-merge was automatically disabled March 17, 2025 10:18

Head branch was pushed to by a user without write access

auto-merge was automatically disabled March 17, 2025 10:18

Pull request was closed

@aziztouhami
Copy link
Copy Markdown
Contributor Author

So, do I have to create another pull request?

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

Labels

PR: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants