Skip to content
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

Trying to play video that is already playing. #33

Closed
bblanchon opened this issue Aug 9, 2020 · 10 comments
Closed

Trying to play video that is already playing. #33

bblanchon opened this issue Aug 9, 2020 · 10 comments

Comments

@bblanchon
Copy link

Hi,

Thank you very much for this awesome library.

I observed the following warning on Chrome 84 but not on Firefox 79:

BrowserCodeReader.js:509 Trying to play video that is already playing.

Steps to reproduce:

Best regards,
Benoit

@github-actions
Copy link

github-actions bot commented Oct 8, 2020

Stale issue message

@bblanchon
Copy link
Author

This issue is not stale: three persons added a thumb up.
Please reopen.

@odahcam
Copy link
Member

odahcam commented Oct 21, 2020

Trying to talk to this bot is useless, trust me, hehe.

You should not be that worried about this console message, but I'll certanly reopen and put this on our backlog as this behavior should not happen.

The message shouldn't be removed, as it helps identifing a problem with the browser layer itself, but some rewriting should be done in the callstack that lead to this point to avoid bad video element handling.

@odahcam odahcam reopened this Oct 21, 2020
@odahcam
Copy link
Member

odahcam commented Oct 21, 2020

Ah, this will probably be transfered to @zxing-js/browser in the near future, as the browser package is finally starting to take some form and is already better tested than in here.

@odahcam odahcam transferred this issue from zxing-js/library Dec 23, 2020
@odahcam
Copy link
Member

odahcam commented Jan 27, 2021

Is this still relevant?

@bblanchon
Copy link
Author

Yes, I still have this warning with 0.18.3

@odahcam
Copy link
Member

odahcam commented Feb 1, 2021

Are you talking about the browser layer present in the library? That browser layer is deprecated and the current browser layer version is v0.0.7.

@bblanchon
Copy link
Author

I used yarn add @zxing/library and imported BrowserCodeReader; this isn't the way to go anymore?

@odahcam
Copy link
Member

odahcam commented Feb 2, 2021

No, actually I have to update the README, I didn't see it was wrong. You should use @zxing/browser for the browser layer.

@odahcam odahcam closed this as completed Feb 2, 2021
@mike-rambil
Copy link

I have this same issue and the camera doesn't start scanning anymore?

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

No branches or pull requests

3 participants