Merge pull request #129 from cynack/feat/want_use_ar_count

feat: PC/MacでAR起動を試みた回数を取得
cynack · May 31, 2022 · 81318e6 · 81318e6
2 parents 67b9693 + a507da2
commit 81318e6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/figni-viewer-base.js b/src/figni-viewer-base.js
@@ -12,6 +12,7 @@ export default class FigniViewerBaseElement extends ModelViewerElement {
   // analytics data
   #websocket
   #arCount = 0
+  #wantUseArCount = 0
   #hotspotClickCount = {}
   #animationPlayCount = {}
   #helpPageViewCount = {}
@@ -134,16 +135,16 @@ export default class FigniViewerBaseElement extends ModelViewerElement {
   }
 
   /**
-   * ARを起動する
+   * ARの起動を試みる
    */
-  activateARMode() {
+  tryActivateAR() {
+    this.#wantUseArCount++
+    startMesure('view-time')
     if (this.canActivateAR) {
       this.#arCount++
       if (this.#arCount == 1) {
         endMesure('initial-ar-use-time')
       }
-      startMesure('view-time')
-      this.activateAR()
     }
   }
 
@@ -362,6 +363,7 @@ export default class FigniViewerBaseElement extends ModelViewerElement {
               initial_model_view_time: this.#initialModelViewTime,
               initial_interaction_time: this.#initialInteractionTime,
               ar_count: this.#arCount,
+              want_use_ar_count: this.#wantUseArCount,
               initial_ar_use_time: this.#initialArUseTime,
               hotspot_click: this.#hotspotClickCount,
               animation_play: this.#animationPlayCount,

diff --git a/src/figni-viewer.js b/src/figni-viewer.js
@@ -1046,6 +1046,7 @@ export default class FigniViewerElement extends HTMLElement {
         }
       })
       this.#arButton.addEventListener('click', () => {
+        this.#figniViewerBase.tryActivateAR()
         if (!this.#figniViewerBase.canActivateAR) {
           this.#showQRCodePanel()
         }