Skip to content

Add more options to default playback rate menu#4880

Merged
FreeTubeBot merged 4 commits intoFreeTubeApp:developmentfrom
msagr:development
Apr 12, 2024
Merged

Add more options to default playback rate menu#4880
FreeTubeBot merged 4 commits intoFreeTubeApp:developmentfrom
msagr:development

Conversation

@msagr
Copy link
Contributor

@msagr msagr commented Apr 5, 2024

Add more options to default playback rate menu

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

closes #4564

Description

This pull request makes the playback rate menu smoother by providing users with more playback options at intervals of 0.05 from 0.25 to 3

Desktop

  • OS: Ubuntu
  • OS Version: 22.04
  • FreeTube version: 0.20.0

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) April 5, 2024 18:53
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Apr 5, 2024
@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Apr 5, 2024

Hi @msagr thank you for taking time to address this issue. Unfortunately you have interpreted the request incorrect.

OP meant that the default playback speed should be adjusted properly.

So when video playback rate interval is set to 0.1 the default playback speed should also be configurable in steps of 0.1 instead of being unchanged and showing steps of 0.25

screen-20240405-224750.mp4

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Apr 5, 2024
auto-merge was automatically disabled April 6, 2024 15:27

Head branch was pushed to by a user without write access

@msagr msagr closed this Apr 6, 2024
auto-merge was automatically disabled April 6, 2024 15:27

Pull request was closed

@msagr msagr reopened this Apr 6, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) April 6, 2024 15:45
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Apr 6, 2024
@msagr
Copy link
Contributor Author

msagr commented Apr 6, 2024

Thanks a lot, I have tried to fix it in this PR. Please let me know if any other issue remains in this.

@BarbzYHOOL

This comment was marked as abuse.

absidue
absidue previously approved these changes Apr 7, 2024
@absidue absidue changed the title Add more options to playback rate menu Add more options to default playback rate menu Apr 7, 2024
auto-merge was automatically disabled April 7, 2024 12:51

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) April 7, 2024 12:51
@absidue absidue self-requested a review April 7, 2024 13:00
Copy link
Member

@absidue absidue left a comment

Choose a reason for hiding this comment

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

Please undo the "Add confirmation pop-up when unsubscribing", as that is unrelated to what you were originally aiming for with this pull request.

@absidue absidue added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Apr 7, 2024
@msagr
Copy link
Contributor Author

msagr commented Apr 7, 2024

Oh, I mistakenly opened it in same PR. Should I make a separate commit removing the code i just committed or there is some other way of undoing it I am not aware of in my limited knowledge.

@PikachuEXE
Copy link
Member

  • Checkout the branch (people normally use a separate branch for a PR instead all into one branch but in this case development
  • Create a branch to save the unrelated changes if you want to save them
  • Reset branch development hard to commit make slider precise as per playback speed
  • Force push

@msagr
Copy link
Contributor Author

msagr commented Apr 10, 2024

@efb4f5ff-1298-471a-8973-3d47447115dc type error resolved.

@PikachuEXE
Copy link
Member

The playback rate seems broken when interval set to 0.1/0.5 (coz 0.25 is minimum value and now 1.0 cannot be selected)

image

auto-merge was automatically disabled April 11, 2024 04:30

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) April 11, 2024 04:30
@msagr
Copy link
Contributor Author

msagr commented Apr 11, 2024

Thanks @PikachuEXE for pointing out the issue. I have tried to resolve it. Please let me know if any other bug remains in it.

PikachuEXE
PikachuEXE previously approved these changes Apr 11, 2024
@efb4f5ff-1298-471a-8973-3d47447115dc
  1. Set Video Playback Rate Interval to 0.1
  2. Try to set Default Playback Rate to 0.1
  3. Observe that Default Playback Rate only goes to a minimum of 0.3
VirtualBoxVM_7a78VAy0o9.mp4

auto-merge was automatically disabled April 11, 2024 11:46

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) April 11, 2024 11:46
@msagr
Copy link
Contributor Author

msagr commented Apr 11, 2024

Done

@msagr msagr requested a review from PikachuEXE April 11, 2024 11:49
Copy link
Member

Choose a reason for hiding this comment

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

Works like a charm, Thanks for the PR!

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels Apr 11, 2024
@FreeTubeBot FreeTubeBot merged commit ae8e0ce into FreeTubeApp:development Apr 12, 2024
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Apr 12, 2024
@Massimo-B
Copy link

Massimo-B commented Oct 24, 2024

What are the current possible values for Video Playback Rate Interval? The smallest is 0.1 here, I'm on v0.21.3 Beta.
Beside the already added 0.1 from #1238 it would be nice to also add 0.05 and 0.01. Using https://github.com/TeamNewPipe/NewPipe, with well cut content, I usually use 1.05 or 1.07 depending on the producer and content.

They use a different UI which would avoid having a setting like that. They have a dialog with slider and step-down/step-up buttons, additionally buttons for setting the step-width like 1% 5% 10% 25% 100%:
Screenshot_2024-10-24-09-38-48-916_org schabi newpipe

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.

[Feature Request]: Make Default playback speed more tunable

8 participants