Skip to content

fix: Playback speed shortcuts and reset speed on new video#798

Merged
PartyDonut merged 5 commits intoDonutWare:developfrom
Julien9969:fix/speed-shortcuts
Mar 4, 2026
Merged

fix: Playback speed shortcuts and reset speed on new video#798
PartyDonut merged 5 commits intoDonutWare:developfrom
Julien9969:fix/speed-shortcuts

Conversation

@Julien9969
Copy link
Contributor

@Julien9969 Julien9969 commented Feb 28, 2026

Pull Request Description

  • Fix keyboard shortcuts crtl + up for speed up not working
  • Reset playbackRateProvider to 1.0 when loading a new playback item to prevent stale speed UI values
  • Remove manual focusNode.requestFocus() that conflicted with autofocus, fixing keyboard-only navigation
  • Fix PathAccessException crash in sync provider temp file cleanup on Windows

Issue Being Fixed

#795

Checklist

  • If a new package was added, did you ensure it works for all supported platforms? Is the package well maintained
  • Check that any changes are related to the issue at hand.

- Fix keyboard shortcuts being replaced when speed boost is active by combining
  onKeyEvent with the default key map handler in InputHandler
- Reset playbackRateProvider to 1.0 when loading a new playback item to prevent
  stale speed UI values
- Remove manual focusNode.requestFocus() that conflicted with autofocus,
  fixing keyboard-only navigation
- Fix PathAccessException crash in sync provider temp file cleanup on Windows
@PartyDonut PartyDonut added the bug Something isn't working label Mar 4, 2026
@PartyDonut PartyDonut changed the title Fix playback speed shortcuts and reset speed on new video fix: Playback speed shortcuts and reset speed on new video Mar 4, 2026
@PartyDonut PartyDonut merged commit b007b73 into DonutWare:develop Mar 4, 2026
1 check passed
@github-project-automation github-project-automation bot moved this to Done in Fladder Mar 4, 2026
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.

2 participants