Custom hook for state management in Transcript component #652
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.
Related issue: #630
This work moves state management in Transcript component to a custom hook.
Since
Transcript
component is built as an independent component from global state management using context providers it doesn't have the issue other components have with unnecessary re-renders.But it can be used within context providers, without breaking its independent nature.
This refactor helps up to make use of the global state IF the component is wrapped inside the context providers.
One advantage of this is,
Transcript
component doesn't need to spin up a unnecessary JS interval to read the player instance from the page when the player is available via context providers, while still supporting its independent implementation.