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(video): add download button and overlay to video player (DEV-1151) #798
Conversation
<mat-icon>more_vert</mat-icon> | ||
</button> | ||
<mat-menu #more="matMenu"> | ||
<button mat-menu-item (click)="openVideoInNewTab(video['changingThisBreaksApplicationSecurity'])"> |
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.
I believe this will break in production.
You'll probably have to use the bypass methods provided by Angular.
We already have access to the file url via this.src.fileValue.fileUrl
so you can just replace video['changingThisBreaksApplicationSecurity']
with that.
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.
done in 6ae945d
|
||
this._http.get(pathToJson, requestOptions).subscribe( | ||
res => { | ||
console.log('a', res); |
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.
these console.logs can be removed
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.
fixed in b852777
@@ -9,6 +9,24 @@ | |||
(canplaythrough)="loadedVideo()"> | |||
</video> | |||
|
|||
<div class="video-overlay" *ngIf="!play && firstPlayed"> |
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.
The video overlay should only show when hovering over the video no matter if it's playing or not. The play button should appear if it's paused and vice versa.
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.
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.
amazing! thank you :)
resolves DEV-1151