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
feat(representation): implement video player incl. preview (DEV-701) #698
Conversation
); | ||
|
||
dialogRef.afterClosed().subscribe((data) => { | ||
this._replaceFile(data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should check if data
is undefined here, otherwise it breaks the video player. This is my fault actually and should be changed in each openReplaceFileDialog
that I added in #695. Interestingly this didn't cause an issue with the other openReplaceFileDialog
methods which is why I didn't catch it. Should I open a separate PR for this once this is merged?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should do it in a separate task.
I can disable the "replace-file" button in this task to avoid crashing player.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved in a6527d4
src/app/workspace/resource/representation/video/video-preview/video-preview.component.scss
Outdated
Show resolved
Hide resolved
src/app/workspace/resource/representation/video/video-preview/video-preview.component.scss
Outdated
Show resolved
Hide resolved
src/app/workspace/resource/representation/video/video-preview/video-preview.component.ts
Outdated
Show resolved
Hide resolved
src/app/workspace/resource/representation/video/video.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks :)
Thank you @mdelez 🎉 |
resolves DEV-701 and DEV-26