Add hotkeys to the audio player #1203
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@RatulMaharaj is attempting to deploy a commit to the Syntax Team on Vercel. A member of the Team first needs to authorize it. |
RatulMaharaj
force-pushed
the
player-hotkeys
branch
2 times, most recently
from
f08a141
to
7e98703
Compare
|
@wesbos this should be ready for review now. |
RatulMaharaj
force-pushed
the
player-hotkeys
branch
from
675790f
to
fc6a223
Compare
|
thanks a ton - will review shortly |
|
This should also fix: #1338 |
|
Love the menu here. @wesbos would you mind if I were to merge this into my embed branch instead of main and merge with the work i'm doing? |
|
Hey @RatulMaharaj this is awesome however since it conflicts with some design changes I've made in a branch, I'm going to merge it there instead of main and massage the merge a bit. Thank you, this is great! |
stolinski
added a commit
that referenced
this pull request
Dec 15, 2023
* adds header icons * Add hotkeys to the audio player (#1203) * Add @svelte-put/shortcut as a dependency * Implement player hotkeys * Pause and reset show on player close * Implement minimized player * Move hotkeys into +layout and refactor into object * Remove unused test file * Implement hotkey modal * Add vitest tests for formatShortcut util * Rebuild lockfile after rebase * Update modal styling * Reset core-js version --------- Co-authored-by: Scott Tolinski <[email protected]> * moves status into it's own writable * Stop the player and reset player state when close is called (#1425) Co-authored-by: Scott Tolinski <[email protected]> * adds more complex player status to player state for more control * reworks hotkeys to fix play pause issues * major work on hotkeys and adds share window * modifies redirects hook to keep query params on redirect * adds completes timestamp at time links * slight tweak to hotkeys * ficxes icon issue caused by merge --------- Co-authored-by: Ratul Maharaj <[email protected]> Co-authored-by: Craig Holliday <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR Implements hotkeys to control the audio player via the keyboard.
imSpacejl<>?I don't have strong opinions about what the actual keys used should be, so if anyone else does, happy for it to be changed.
Related to #1195
Updates
Stopping audio and removing show on player close.
Implemented a
MINIMIZEDversion of the playerAdded
?hotkey to the aboveAdded a global hotkey
?/shift+/to bring up the list of hotkeys.