From 016b75c8f0474cda40805029b6cd886c537b9595 Mon Sep 17 00:00:00 2001
From: "Elevator Dev (ubuntu)" <latistecharch@umn.edu>
Date: Fri, 7 Jul 2023 03:54:20 +0000
Subject: [PATCH] Move transition to tab contents

There's a bug that appears when building for production if Transition wraps an element without a `v-if`.

https://github.com/vuejs/core/issues/5657
---
 src/pages/DrawerViewPage/DrawerViewPage.vue | 140 +++++++++++---------
 1 file changed, 74 insertions(+), 66 deletions(-)

diff --git a/src/pages/DrawerViewPage/DrawerViewPage.vue b/src/pages/DrawerViewPage/DrawerViewPage.vue
index bb566cbf..c10bb144 100644
--- a/src/pages/DrawerViewPage/DrawerViewPage.vue
+++ b/src/pages/DrawerViewPage/DrawerViewPage.vue
@@ -1,65 +1,65 @@
 <template>
   <DefaultLayout class="drawer-view-page">
-    <Transition name="fade">
-      <div class="px-4">
-        <header class="my-8">
-          <Link
-            :to="`/drawers/listDrawers`"
-            class="flex items-center gap-1 mb-4 hover:no-underline"
-          >
-            <ArrowForwardIcon class="transform rotate-180 h-4 w-4" />
-            Back to Drawers
-          </Link>
-          <h2 class="text-4xl font-bold">{{ drawerTitle }}</h2>
-        </header>
+    <div class="px-4">
+      <header class="my-8">
+        <Link
+          :to="`/drawers/listDrawers`"
+          class="flex items-center gap-1 mb-4 hover:no-underline"
+        >
+          <ArrowForwardIcon class="transform rotate-180 h-4 w-4" />
+          Back to Drawers
+        </Link>
+        <h2 class="text-4xl font-bold">{{ drawerTitle }}</h2>
+      </header>
 
-        <Tabs
-          labelsClass="drawer-view-page__tabs sticky top-14 z-20  -mx-4 px-4 border-b border-neutral-200 pt-4"
-          :activeTabId="activeTabId"
-          @tabChange="handleTabChange"
+      <Tabs
+        labelsClass="drawer-view-page__tabs sticky top-14 z-20  -mx-4 px-4 border-b border-neutral-200 pt-4"
+        :activeTabId="activeTabId"
+        @tabChange="handleTabChange"
+      >
+        <div
+          class="bg-transparent-black-50 rounded-md mb-4 sm:flex justify-between items-center p-2"
         >
-          <div
-            class="bg-transparent-black-50 rounded-md mb-4 sm:flex justify-between items-center p-2"
-          >
-            <div>
-              <ResultsCount
-                v-if="drawer.contents"
-                class="mb-2 sm:mb-0"
-                :total="drawer.contents.totalResults"
-                :fetchStatus="fetchStatus"
-                :showingCount="drawer.contents.matches.length"
-                @loadMore="handleLoadMore"
-                @loadAll="handleLoadAll"
-              />
+          <div>
+            <ResultsCount
+              v-if="drawer.contents"
+              class="mb-2 sm:mb-0"
+              :total="drawer.contents.totalResults"
+              :fetchStatus="fetchStatus"
+              :showingCount="drawer.contents.matches.length"
+              @loadMore="handleLoadMore"
+              @loadAll="handleLoadAll"
+            />
 
-              <div
-                v-else
-                class="flex items-center gap-2 text-neutral-500 text-sm"
-              >
-                <SpinnerIcon class="animate-spin h-5 w-5" />
-                Loading...
-              </div>
+            <div
+              v-else
+              class="flex items-center gap-2 text-neutral-500 text-sm"
+            >
+              <SpinnerIcon class="animate-spin h-5 w-5" />
+              Loading...
             </div>
-            <div class="flex items-baseline gap-2">
-              <label for="location" class="sr-only">Sort</label>
-              <select
-                id="sort"
-                name="sort"
-                class="block w-full rounded-md border-0 py-1.5 pl-3 pr-10 text-neutral-900 ring-1 ring-inset ring-neutral-300 focus:ring-2 focus:ring-indigo-600 text-sm sm:leading-6 max-w-full bg-transparent-white-800"
-                :value="selectedSortOption"
-                @change="handleSortOptionChange"
+          </div>
+          <div class="flex items-baseline gap-2">
+            <label for="location" class="sr-only">Sort</label>
+            <select
+              id="sort"
+              name="sort"
+              class="block w-full rounded-md border-0 py-1.5 pl-3 pr-10 text-neutral-900 ring-1 ring-inset ring-neutral-300 focus:ring-2 focus:ring-indigo-600 text-sm sm:leading-6 max-w-full bg-transparent-white-800"
+              :value="selectedSortOption"
+              @change="handleSortOptionChange"
+            >
+              <option
+                v-for="(sortOptionLabel, sortOptionKey) in sortOptions"
+                :key="sortOptionKey"
+                :value="sortOptionKey"
               >
-                <option
-                  v-for="(sortOptionLabel, sortOptionKey) in sortOptions"
-                  :key="sortOptionKey"
-                  :value="sortOptionKey"
-                >
-                  {{ sortOptionLabel }}
-                </option>
-              </select>
-            </div>
+                {{ sortOptionLabel }}
+              </option>
+            </select>
           </div>
-          <Tab id="grid" label="Grid">
+        </div>
+        <Tab id="grid" label="Grid">
+          <Transition name="fade">
             <DrawerItemsGrid
               v-if="drawer.contents"
               :totalResults="drawer.contents.totalResults"
@@ -69,8 +69,10 @@
               :isDraggable="selectedSortOption === SORT_KEYS.CUSTOM"
               @dragEnd="handleDragEnd"
             />
-          </Tab>
-          <Tab id="list" label="List">
+          </Transition>
+        </Tab>
+        <Tab id="list" label="List">
+          <Transition name="fade">
             <DrawerItemsList
               v-if="drawer.contents"
               :totalResults="drawer.contents.totalResults"
@@ -80,34 +82,40 @@
               :isDraggable="selectedSortOption === SORT_KEYS.CUSTOM"
               @dragEnd="handleDragEnd"
             />
-          </Tab>
-          <Tab id="timeline" label="Timeline">
+          </Transition>
+        </Tab>
+        <Tab id="timeline" label="Timeline">
+          <Transition name="fade">
             <SearchResultsTimeline
               v-if="drawer.contents"
               :totalResults="drawer.contents.totalResults"
               :matches="drawer.contents.matches"
               :status="fetchStatus"
             />
-          </Tab>
-          <Tab id="map" label="Map">
+          </Transition>
+        </Tab>
+        <Tab id="map" label="Map">
+          <Transition name="fade">
             <SearchResultsMap
               v-if="drawer.contents"
               :totalResults="drawer.contents.totalResults"
               :matches="drawer.contents.matches"
               :status="fetchStatus"
             />
-          </Tab>
-          <Tab id="gallery" label="Gallery">
+          </Transition>
+        </Tab>
+        <Tab id="gallery" label="Gallery">
+          <Transition name="fade">
             <SearchResultsGallery
               v-if="drawer.contents"
               :totalResults="drawer.contents.totalResults"
               :matches="drawer.contents.matches"
               :status="fetchStatus"
             />
-          </Tab>
-        </Tabs>
-      </div>
-    </Transition>
+          </Transition>
+        </Tab>
+      </Tabs>
+    </div>
   </DefaultLayout>
 </template>
 <script setup lang="ts">