Fixed error in live_example; Working for iOS in Safari 11 beta

serratus · Jun 7, 2017 · e8352c7 · e8352c7
1 parent 74ed8c0
commit e8352c7
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -1,7 +1,7 @@
 quaggaJS
 ========
 
-- [Changelog](#changelog) (2017-06-06)
+- [Changelog](#changelog) (2017-06-07)
 - [Browser Support](#browser-support)
 - [Installing](#installing)
 - [Getting Started](#gettingstarted)
@@ -703,6 +703,13 @@ on the ``singleChannel`` flag in the configuration when using ``decodeSingle``.
 
 ## <a name="changelog">Changelog</a>
 
+### 2017-06-07
+- Improvements
+  - added `muted` and `playsinline` to `<video/>` to make it work for Safari 11
+  Beta (even iOS)
+- Fixes
+  - Fixed  [example/live_w_locator.js](https://github.com/serratus/quaggaJS/blob/master/example/live_w_locator.js)
+
 ### 2017-06-06
 - Features
   - Support for Standard 2of5 barcodes (See

diff --git a/dist/quagga.js b/dist/quagga.js
diff --git a/dist/quagga.min.js b/dist/quagga.min.js
diff --git a/example/live_w_locator.js b/example/live_w_locator.js
@@ -81,7 +81,6 @@ $(function() {
                 }
                 return;
             }
-            node.parentNode.style.display = 'none';
         },
         initCameraSelection: function(){
             var streamLabel = Quagga.CameraAccess.getActiveStreamLabel();

diff --git a/lib/quagga.js b/lib/quagga.js
diff --git a/src/input/camera_access.js b/src/input/camera_access.js
@@ -42,7 +42,9 @@ function initCamera(video, constraints) {
     .then((stream) => {
         return new Promise((resolve) => {
             streamRef = stream;
-            video.setAttribute("autoplay", 'true');
+            video.setAttribute("autoplay", true);
+            video.setAttribute('muted', true);
+            video.setAttribute('playsinline', true);
             video.srcObject = stream;
             video.addEventListener('loadedmetadata', () => {
                 video.play();