Skip to content

Add keyboard shortcuts to jump to the next and previous chapters #2912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 12, 2022

Conversation

absidue
Copy link
Member

@absidue absidue commented Dec 4, 2022

Add keyboard shortcuts to jump to the next and previous chapters

Pull Request Type

  • Feature Implementation

Related issue

closes #2823

Description

This pull request adds keyboard shortcuts to jump to the next and previous chapters using Option+left arrow and Option+right arrow on macOS and Ctrl+left arrow and Ctrl+right arrow on Windows and Linux, these are the same as on YouTube.

Testing

Watch a video with chapters, e.g. https://youtu.be/OHKKcd3sx2c and test the keyboard shortcuts

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.18.0

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) December 4, 2022 15:40
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 4, 2022
@ChunkyProgrammer
Copy link
Member

Could you open a pr on freetubedocs as well?

@PikachuEXE
Copy link
Collaborator

What are the keys ._.
image

@ChunkyProgrammer
Copy link
Member

What are the keys ._. image

It should be ctrl + Arrow left or ctrl + arrow right

@PikachuEXE
Copy link
Collaborator

Testing chars:
Option + < and Option + >
What char did @absidue use to break the display ❓

@PikachuEXE
Copy link
Collaborator

Found a problem:
Option + < and Option + > = Prev/Next Page on electron ._.
Pressed Option + <, navigated to previous page

@absidue
Copy link
Member Author

absidue commented Dec 5, 2022

@PikachuEXE looks like Apple OS's don't display those unicode arrow characters.

@absidue
Copy link
Member Author

absidue commented Dec 5, 2022

written out the keys now

@PikachuEXE
Copy link
Collaborator

Prev/Next Page on electron problem still valid ._.

@absidue
Copy link
Member Author

absidue commented Dec 5, 2022

@PikachuEXE
Copy link
Collaborator

How about using Cmd+left arrow and Cmd+right arrow?

@absidue
Copy link
Member Author

absidue commented Dec 10, 2022

Updated the shortcuts code

Copy link
Collaborator

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

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

Tested MacOS, not tested windows

Update: tested on windows too

@FreeTubeBot FreeTubeBot merged commit 10f9bf6 into FreeTubeApp:development Dec 12, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Dec 12, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc

Im unable atm to create a PR in the docs repo for this. Could somebody open up a PR to add this shortcut to the docs :)

@PikachuEXE
Copy link
Collaborator

FreeTubeApp/FreeTube-Docs#74

Just do (approve) it
image

@DnickC
Copy link

DnickC commented Oct 30, 2023

Has this been tested on linux? I'm on mint using release 19.1 and this does not work for me.

@efb4f5ff-1298-471a-8973-3d47447115dc

It worked on Ubuntu but maybe something broke it?

@kommunarr
Copy link
Collaborator

This is still working for me on OpenSUSE. Do you have a particular video link this isn't working for? Also make sure your focus isn't on a particular button, otherwise you may be encountering #4137.

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]: Add missing keyboard shortcuts to navigate chapters: "CONTROL + ←", "CONTROL + →"
7 participants