-
Notifications
You must be signed in to change notification settings - Fork 43
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
Subtitle Editor #638
Subtitle Editor #638
Conversation
Any functionality of the edit view components is purely coincidental
First implementation of a list view for subtitle editing. Does not contain any functionality yet.
- Adds a redux slice for the subtitle editor view - Generalizes some redux selector and dispatch functions in Timeline.tsx to props. This should prepare the timeline component to work independently from the cutting view.
Need to look for a working parser
Does NOT do anything yet, it's just UI
This pull request has conflicts ☹ |
Are you gonna squash those commits before merging? :) |
This pull request is deployed at test.editor.opencast.org/638/2022-10-06_15-23-09/ . |
Not planning to. While it's a hell of a lot of commits, most of them actually do relevant stuff, and squashing them all together feels like grossly disregarding the development history. |
This pull request is deployed at test.editor.opencast.org/638/2022-10-07_11-57-50/ . |
This pull request is deployed at test.editor.opencast.org/638/2022-10-07_15-21-49/ . |
This pull request has conflicts ☹ |
This pull request is deployed at test.editor.opencast.org/638/2022-10-12_14-55-29/ . |
opencast-editor-html.mp4 |
Thanks, now I can reproduce it too :) |
As discussed in Matrix, I'm pretty happy with the current state of the new subtitle editor. The minor problem with the potential double-encoding of HTML content in subtitles, which is a somewhat obscure problem normal users are unlikely to face I'm going to ignore here. |
Includes all previous PRs for the subtitle editor.
This PR aims at integrating the different views of the subtitle editor with the parsed subtitles from Opencast.