Bug hunt 3 (#564)

* Fix applyPreset, don't use updateFromRange to avoid infinite loop * Ensure user is defined before rendering app (in response to Sentry event 1a573fba85db4fb1a78735051daa3317)
OpenImaging · Aug 16, 2022 · 61cce38 · 61cce38
1 parent be8391d
commit 61cce38
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/web_client/src/components/WindowWidget.vue b/web_client/src/components/WindowWidget.vue
@@ -76,7 +76,10 @@ export default defineComponent({
       currentRange.value = windowPresets.find(
         (preset) => preset.value === presetId,
       ).apply(widthMin.value, widthMax.value);
-      updateFromRange(currentRange.value);
+      const [v0, v1] = currentRange.value;
+      const ww = v1 - v0;
+      const wl = v0 + Math.floor(ww / 2);
+      updateRender(ww, wl);
     }
 
     onMounted(() => {

diff --git a/web_client/src/views/Projects.vue b/web_client/src/views/Projects.vue
@@ -149,7 +149,7 @@ export default defineComponent({
 </script>
 
 <template>
-  <div>
+  <div v-if="user">
     <Navbar />
     <div class="d-flex">
       <v-card class="project-list-container">