Skip to content

Black screen on some mobile chrome and safari#99

Open
yamaha252 wants to merge 2 commits intobasst314:masterfrom
yamaha252:95
Open

Black screen on some mobile chrome and safari#99
yamaha252 wants to merge 2 commits intobasst314:masterfrom
yamaha252:95

Conversation

@yamaha252
Copy link
Contributor

Fix #95

/** MediaStream object in use for streaming UserMedia data */
private mediaStream: MediaStream = null;
@ViewChild('video', { static: true }) private video: any;
@ViewChild('video', {static: true}) private video: any;
Copy link
Contributor Author

@yamaha252 yamaha252 Nov 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry for that changes

@jmeinke
Copy link

jmeinke commented Dec 22, 2020

Thanks for the fix @yamaha252 - we are facing the problem mentioned in #95 as well.
@basst314 It would be great if you could merge this and release a new version of the library!

@jmeinke
Copy link

jmeinke commented Dec 22, 2020

@yamaha252 Could you explain what the problem was? Why are your code changes fixing the problem?

@yamaha252
Copy link
Contributor Author

It seems twice execution of navigator.mediaDevices.enumerateDevices causes this issue. I changed WebcamUtil.getAvailableVideoInputs method to avoid execution if there are already video inputs. That's the main change.

@jmeinke
Copy link

jmeinke commented Jan 8, 2021

@basst314 Happy new year to you - it would be great if you could merge this sometime soon. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Black screen on live-demo

2 participants