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

Skip prevented keyboard events #6496

Merged

Conversation

dmitrylyzo
Copy link
Contributor

Changes

  • Prevent default actions for hotkeys when OSD is hidden.
  • Skip prevented keyboard events.

Issues
This doesn't fix jellyfin/jellyfin-tizen#331 (comment), but we should skip the prevented keyboard event anyway.

@dmitrylyzo dmitrylyzo added bug Something isn't working stable backport Backport into the next stable release labels Jan 31, 2025
@dmitrylyzo dmitrylyzo added this to the v10.10.6 milestone Jan 31, 2025
@dmitrylyzo dmitrylyzo requested a review from a team as a code owner January 31, 2025 11:36
@jellyfin-bot
Copy link
Collaborator

jellyfin-bot commented Jan 31, 2025

Cloudflare Pages deployment

Latest commit 8456317
Status ✅ Deployed!
Preview URL https://e2b38c25.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

@thornbill thornbill merged commit 5d8ab6a into jellyfin:release-10.10.z Jan 31, 2025
15 checks passed
@dmitrylyzo dmitrylyzo deleted the skip-prevented-keyboard-event branch January 31, 2025 17:24
joshuaboniface pushed a commit that referenced this pull request Feb 3, 2025
Skip prevented keyboard events

Original-merge: 5d8ab6a

Merged-by: thornbill <[email protected]>

Backported-by: Joshua M. Boniface <[email protected]>
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants