Skip to content
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

Fix the trending and watch pages swallowing the arrow key events #3170

Merged
merged 2 commits into from
Feb 10, 2023

Conversation

absidue
Copy link
Member

@absidue absidue commented Feb 7, 2023

Fix the trending and watch pages swallowing the arrow key events

Pull Request Type

  • Bugfix

Related issue

closes #3160

Description

Currently the trending and watch pages swallow the arrow key events, even when you use the alt+arrow key shortcuts.

Testing

Focus the different trending tabs with the left and right arrow keys and also try navigating back and forward with alt+arrow key

Desktop

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

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 7, 2023
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 7, 2023 22:00
PikachuEXE
PikachuEXE previously approved these changes Feb 8, 2023
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.

image

Copy link
Member

Choose a reason for hiding this comment

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

Video page still swallows the arrow keys :(

@ChunkyProgrammer
Copy link
Member

Video page still swallows the arrow keys :(

I think this or is just for fixing the trending page

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

Video page still swallows the arrow keys :(

I think this or is just for fixing the trending page

Ah if that is the case i can approve it but this pr doesnt fully close the issue.

@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 Feb 8, 2023
@absidue absidue changed the title Fix the trending page swallowing the arrow key events Fix the trending and watch page swallowing the arrow key events Feb 9, 2023
@absidue absidue changed the title Fix the trending and watch page swallowing the arrow key events Fix the trending and watch pages swallowing the arrow key events Feb 9, 2023
@absidue absidue added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels Feb 9, 2023
Copy link
Member

Choose a reason for hiding this comment

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

lgtm

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 video watch view

@FreeTubeBot FreeTubeBot merged commit a26c8e0 into FreeTubeApp:development Feb 10, 2023
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 10, 2023
@absidue absidue deleted the trending-navigation branch February 10, 2023 17:44
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.

[Bug]: Shortcut alt + -> / <- doesnt work on trending and video page
5 participants