Make Mark Step button immediately start next step; Start in Step 1

hobinjk-ptc · Apr 4, 2024 · 6eda915 · 6eda915
1 parent f7f38ce
commit 6eda915
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 21 deletions.
diff --git a/tools/spatialAnalytics/index.css b/tools/spatialAnalytics/index.css
@@ -92,13 +92,6 @@ img {
     margin-top: calc(50% - 19px);
 }
 
-.iconBackground.start {
-    border-color: green;
-}
-.iconBackground.end {
-    border-color: red;
-}
-
 .markStepIcon.pressed {
     filter: brightness(0.5);
 }

diff --git a/tools/spatialAnalytics/index.js b/tools/spatialAnalytics/index.js
@@ -82,6 +82,7 @@ recordingIcon.addEventListener('pointerup', function() {
     case RecordingState.empty:
         setRecordingState(RecordingState.recording);
         startTime = Date.now();
+        regionCardStartTime = startTime;
         spatialInterface.analyticsSetDisplayRegion({
             recordingState,
             startTime,
@@ -157,20 +158,13 @@ markStepIcon.addEventListener('pointerup', function() {
     if (recordingState !== RecordingState.recording) {
         return;
     }
-    if (regionCardStartTime > 0) {
-        let regionCardEndTime = Date.now();
-        appendRegionCard({
-            startTime: regionCardStartTime,
-            endTime: regionCardEndTime,
-        });
-        regionCardStartTime = -1;
-        markStepIcon.parentNode.classList.remove('end');
-        markStepIcon.parentNode.classList.add('start');
-    } else {
-        regionCardStartTime = Date.now();
-        markStepIcon.parentNode.classList.remove('start');
-        markStepIcon.parentNode.classList.add('end');
-    }
+    let regionCardEndTime = Date.now();
+    appendRegionCard({
+        startTime: regionCardStartTime,
+        endTime: regionCardEndTime,
+    });
+    // Immediately start next regionCard
+    regionCardStartTime = regionCardEndTime;
 });
 
 let lastSetDisplayRegion = {};