nkoehler / mat-video

:tv: mat-video is an Angular 8/9+ video player using Material!
https://nkoehler.github.io/mat-video/
MIT License
91 stars 47 forks source link

[BUG] Extra unnecessary src="null" in the rendered <video> element #59

Open vbearn opened 4 years ago

vbearn commented 4 years ago

Bug, feature request, or proposal:

When using the <sources> tag for adding video sources, there is an extra src="null" attribute In the rendered DOMElements for the <video> tag, which leads to the failure of the loading of the video (I get an infinite loading spinner)

What is the current behavior?

The extra src="null" attribute IS rendered image

What is the expected behavior?

The extra src="null" attribute SHOULD NOT BE rendered

What are the steps to reproduce?

Happens in this stackblitz: https://stackblitz.com/edit/angular-p1cgj5 File: app.component.html

What is the use-case or motivation for changing an existing behavior?

-

Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Angular/Material 9+ mat-video 2.8.1

Is there anything else I should know?

-