Ensure media is always visible and segments are followed dynamically.#3
Merged
Ensure media is always visible and segments are followed dynamically.#3
Conversation
Owner
|
Thanks for this @markpbaggett -- and my sincere apologies for not seeing your PR sooner. I was wondering, did you try overriding the CSS rules in your own application that is using whisper-transcript? |
Owner
|
We followed up in Slack and it looks like LitElement's use of the Shadow DOM makes it difficult to override CSS. I'm going to look into how people deal with this, perhaps having options for the CSS is the way to go? For example: <whisper-transcript audio="media.mp3" url="media.json" height=400 sticky></whisper-transcript> |
Owner
|
I merged these changes, with the exception of not hard coding the <whisper-transcript maxHeight="400px" audio="media.mp3" url="media.json"></whisper-transcript>I figure people will want to control that based on where the component is being used. It has been released to NPM as v0.1.1. |
This file contains hidden or 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
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.
What Does This Do
This is a minor change that makes the player a little more user friendly.
First, the audio canvas is always sticky so it never is not visible no matter where the media is currently.
Second, it keeps segments and the media player always in sync so that as the current timestamp is updated the segment is always visible in the middle of the player.
Why am I adding this?
We've started to use this at my current shop to help in our process of reviewing vtt and transcript files. This adds functionality so I don't need to fork this or write my own thing from scratch.
If this is unwanted, no problem.