Skip to content

Conversation

@JamesBotterill
Copy link
Contributor

⚠️ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠️

What kind of change does this PR introduce?

New PR on correct branch.

Adding a event to stop and reset video on marker lost within videohandler

Can it be referenced to an Issue? If so what is the issue # ?

How can we test it?

Scan a marker with video playing, move away from marker so video disappears and then rescan marker

Summary

This is purely to expand on the videohandler component to handle the video being lost to stop and reset

Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser

@nicolocarpignoli
Copy link
Member

It's ok! Thank you for contribute :)

@nicolocarpignoli nicolocarpignoli merged commit c487d3c into AR-js-org:dev Mar 20, 2020
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.

2 participants