Skip to content
This repository has been archived by the owner on Jul 31, 2020. It is now read-only.

Cannot set property 'muted' of null #45

Closed
peavers opened this issue Mar 21, 2020 · 8 comments
Closed

Cannot set property 'muted' of null #45

peavers opened this issue Mar 21, 2020 · 8 comments

Comments

@peavers
Copy link

peavers commented Mar 21, 2020

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
Copy link

Also getting this exact behaviour

@buu700 buu700 mentioned this issue Mar 26, 2020
@AndriusdevLa
Copy link

Also getting cannot get muted of null

@chedydy
Copy link

chedydy commented Mar 30, 2020

same here

@born2net
Copy link

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

@MichaelShi1979
Copy link

what's the status of this bug?

@peavers
Copy link
Author

peavers commented Mar 31, 2020

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

@born2net
Copy link

born2net commented Apr 3, 2020

when can it be merged? tx!

@nkoehler
Copy link
Owner

nkoehler commented Apr 3, 2020

Fixed in version 2.8.0.

@nkoehler nkoehler closed this as completed Apr 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants