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

Cannot set property 'muted' of null #45

Closed peavers closed 4 years ago

peavers commented 4 years ago

Bug

What is the current behavior?

When loading in a video with a simple configuration such as

        <mat-video
          src="{{ video.absolutePath | remoteVideo }}"
          [download]="true"
          [title]="video.name"
          [poster]="coverImage(video) | remoteImage"
        ></mat-video>

The following error is thrown in the console.

core.js:5845 ERROR TypeError: Cannot set property 'muted' of null
    at MatVolumeControlComponent.set muted [as muted] (mat-video.js:1150)
    at setInputsForProperty (core.js:13504)
    at elementPropertyInternal (core.js:12319)
    at Module.ɵɵproperty (core.js:20842)
    at MatVideoComponent_div_8_Template (mat-video.js:96)
    at executeTemplate (core.js:11858)
    at refreshView (core.js:11705)
    at refreshDynamicEmbeddedViews (core.js:13070)
    at refreshView (core.js:11728)
    at refreshComponent (core.js:13145)

The video however does play and respond still as expected.

What is the expected behavior?

No console error is thrown.

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

"@angular/compiler": "~9.0.1",
"@angular/material": "^9.0.0",
"mat-video": "^2.7.2",
"typescript": "~3.7.5"

Chrome Version 80.0.3987.132 Mac OS 10.15.3

then3rdman commented 4 years ago

Also getting this exact behaviour

AndriusdevLa commented 4 years ago

Also getting cannot get muted of null

chedydy commented 4 years ago

same here

born2net commented 4 years ago

yep... same bug.. started appearing in last version... awesome project, please fix... TX!!!!

MichaelShi1979 commented 4 years ago

what's the status of this bug?

peavers commented 4 years ago

PR is open with a fix; just needs to be merged.

born2net commented 4 years ago

when can it be merged? tx!

nkoehler commented 4 years ago

Fixed in version 2.8.0.