Add note about order of click handlers

google · Aug 7, 2019 · 7a96e82 · 7a96e82
1 parent 8ef495c
commit 7a96e82
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/features/ar.ts b/src/features/ar.ts
@@ -313,6 +313,11 @@ configuration or device capabilities');
 
           if (showArButton) {
             this[$arButtonContainer].classList.add('enabled');
+            // NOTE(cdata): The order of the two click handlers on the "ar
+            // button container" is important, vital to the workaround described
+            // earlier in this file. Reversing their order will cause our Scene
+            // Viewer integration to break.
+            // @see https://github.com/GoogleWebComponents/model-viewer/issues/693
             this[$arButtonContainer].addEventListener(
                 'click', this[$arButtonContainerClickHandler]);
             this[$arButtonContainer].addEventListener(