From b12cdc7658500e8da0aedace1195c76b652fa548 Mon Sep 17 00:00:00 2001
From: Lucas Vogel <contact@vogelino.com>
Date: Wed, 9 Aug 2023 15:54:22 +0200
Subject: [PATCH] refactor(FiltersList): Add loading indication when loading
 search results

---
 pages/map.tsx                      |  3 +-
 src/components/FiltersList.tsx     | 52 ++++++++++++++++++------------
 src/components/FiltersTagsList.tsx |  3 ++
 src/components/Label.tsx           | 20 ++++++++----
 src/components/Listbox.tsx         | 16 ++++++---
 src/components/TextSearch.tsx      |  5 +++
 src/components/icons/Spinner.tsx   | 27 ++++++++++++++++
 src/lib/UrlStateContext.tsx        | 11 ++++++-
 8 files changed, 102 insertions(+), 35 deletions(-)
 create mode 100644 src/components/icons/Spinner.tsx

diff --git a/pages/map.tsx b/pages/map.tsx
index b431c7ed..ac8dbc81 100644
--- a/pages/map.tsx
+++ b/pages/map.tsx
@@ -103,6 +103,7 @@ const MapPage: Page<MapProps> = ({ records: originalRecords }) => {
     [getDistanceToUser, useGeolocation, defaultSort, sortByTagsCount]
   )
 
+  const tagsKey = urlState.tags?.join('-') || ''
   useEffect(() => {
     const filteredRecords = getFilteredFacilities({
       facilities: originalRecords,
@@ -111,7 +112,7 @@ const MapPage: Page<MapProps> = ({ records: originalRecords }) => {
     })
     return setFilteredRecords(sortFacilities(filteredRecords))
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [urlState.tags?.join('-'), activeIdsBySearchTerm.key])
+  }, [tagsKey, activeIdsBySearchTerm.key, activeIdsBySearchTerm.isLoading])
 
   return (
     <>
diff --git a/src/components/FiltersList.tsx b/src/components/FiltersList.tsx
index 1fa23d7b..cec4f9e9 100644
--- a/src/components/FiltersList.tsx
+++ b/src/components/FiltersList.tsx
@@ -20,6 +20,7 @@ import { Button } from './Button'
 import { Arrow } from './icons/Arrow'
 import TextSearch from './TextSearch'
 import { useActiveIdsBySearchTerm } from '@lib/hooks/useActiveIdsBySearchTerm'
+import { Spinner } from './icons/Spinner'
 
 export const FiltersList: FC<{
   recordsWithOnlyLabels: RecordsWithOnlyLabelsType[]
@@ -44,6 +45,7 @@ export const FiltersList: FC<{
     recordsWithOnlyLabels
   )
   const { isLoading: textSearchLoading, total } = useActiveIdsBySearchTerm()
+  const fieldsDisabled = textSearchLoading
 
   const group1 = labels.filter(({ fields }) => fields.group2 === 'gruppe-1')
   const group2 = labels.filter(({ fields }) => fields.group2 === 'gruppe-2')
@@ -57,21 +59,17 @@ export const FiltersList: FC<{
     .map((label) => label.id)
 
   const [activeTargetGroupId, setActiveTargetGroupId] = useState(
-    queryTagIds.find((tagId) => {
-      return targetGroupIds.includes(tagId)
-    })
+    queryTagIds.find((tagId) => targetGroupIds.includes(tagId)) || null
   )
 
+  const tagsKey = queryTagIds.join('-')
   useEffect(() => {
-    if (queryTagIds.length === 0) return
-
     const currentTargetGroupId = queryTagIds.find((tagId) =>
       targetGroupIds.includes(tagId)
     )
-    if (currentTargetGroupId) {
-      setActiveTargetGroupId(currentTargetGroupId)
-    }
-  }, [queryTagIds, targetGroupIds])
+    setActiveTargetGroupId(currentTargetGroupId || null)
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [tagsKey, targetGroupIds])
 
   const someGroupFiltersActive = labels
     .filter(({ fields }) => fields.group2 !== 'zielpublikum')
@@ -88,10 +86,12 @@ export const FiltersList: FC<{
           <FiltersTagsList
             filters={[...group1, ...group2, ...group3]}
             onLabelClick={updateFilters}
+            disabled={fieldsDisabled}
           />
         </ul>
         {someGroupFiltersActive && (
           <button
+            disabled={fieldsDisabled}
             onClick={() =>
               updateFilters(
                 queryTagIds.filter((f) => {
@@ -102,9 +102,13 @@ export const FiltersList: FC<{
             }
             className={classNames(
               `text-lg leading-6 text-left font-normal mb-8`,
-              `focus:outline-none focus:ring-2 focus:ring-primary`,
-              `focus:ring-offset-2 focus:ring-offset-white`,
-              `underline text-gray-80 hover-primary transition-colors`
+              !fieldsDisabled && [
+                `focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-primary`,
+                `focus-visible:ring-offset-2 focus-visible:ring-offset-white`,
+                `underline text-gray-80 hover-primary`,
+              ],
+              fieldsDisabled && `text-gray-40`,
+              `transition motion-reduce:transition-none`
             )}
           >
             {texts.reset}
@@ -126,10 +130,12 @@ export const FiltersList: FC<{
           categories={urlSearchCategoriesToStateSearchCategories(
             urlState.qCategories
           )}
+          disabled={fieldsDisabled}
         />
         <div className="block w-full @md:w-[324px] z-10">
           <Listbox
             label={texts.filtersSearchTargetLabel}
+            disabled={fieldsDisabled}
             options={targetGroups
               .sort((a, b) => {
                 if (!a.fields.order) return 1
@@ -164,14 +170,14 @@ export const FiltersList: FC<{
                   break
                 case targetGroupAlreadyInUrl && !hasValidTargetGroup:
                   updateFilters([...tagsWithoutOldTargetGroup])
-                  setActiveTargetGroupId(undefined)
+                  setActiveTargetGroupId(null)
                   break
                 case !targetGroupAlreadyInUrl && hasValidTargetGroup:
                   updateFilters([...queryTagIds, selectedValue as number])
                   break
                 case !targetGroupAlreadyInUrl && !hasValidTargetGroup:
                   updateFilters([...queryTagIds])
-                  setActiveTargetGroupId(undefined)
+                  setActiveTargetGroupId(null)
                   break
                 default:
                   break
@@ -185,7 +191,7 @@ export const FiltersList: FC<{
       <SwitchButton
         value={useGeolocation}
         onToggle={setGeolocationUsage}
-        disabled={geolocationIsForbidden}
+        disabled={geolocationIsForbidden || fieldsDisabled}
         tooltip={geolocationIsForbidden ? texts.geolocationForbidden : ``}
       >
         {texts.filtersGeoSearchLabel}
@@ -205,13 +211,17 @@ export const FiltersList: FC<{
           })
         }}
         icon={
-          <Arrow
-            className={classNames(
-              'transition-transform group-hover:translate-x-0.5 group-disabled:group-hover:translate-x-0'
-            )}
-          />
+          textSearchLoading ? (
+            <Spinner className={classNames('animate-spin')} />
+          ) : (
+            <Arrow
+              className={classNames(
+                'transition-transform group-hover:translate-x-0.5 group-disabled:group-hover:translate-x-0'
+              )}
+            />
+          )
         }
-        disabled={filteredFacilitiesCount === 0}
+        disabled={filteredFacilitiesCount === 0 || fieldsDisabled}
         tooltip={
           filteredFacilitiesCount === 0 && (
             <span>{texts.filtersButtonTextFilteredNoResultsHint}</span>
diff --git a/src/components/FiltersTagsList.tsx b/src/components/FiltersTagsList.tsx
index 4ebbdb4c..4ec9fe0f 100644
--- a/src/components/FiltersTagsList.tsx
+++ b/src/components/FiltersTagsList.tsx
@@ -7,12 +7,14 @@ interface FiltersTagsListPropType {
   onLabelClick?: (activeFilters: number[]) => void
   filters: FiltersWithActivePropType[]
   className?: string
+  disabled?: boolean
 }
 
 export const FiltersTagsList: FC<FiltersTagsListPropType> = ({
   onLabelClick,
   filters,
   className,
+  disabled = false,
 }) => {
   const [urlState] = useUrlState()
   const activeFilters = urlState.tags || []
@@ -21,6 +23,7 @@ export const FiltersTagsList: FC<FiltersTagsListPropType> = ({
     <>
       {filters.map((filter) => (
         <Label
+          disabled={disabled}
           label={filter}
           key={filter.id}
           isActive={filter.isActive}
diff --git a/src/components/Label.tsx b/src/components/Label.tsx
index c69ae778..f690762e 100644
--- a/src/components/Label.tsx
+++ b/src/components/Label.tsx
@@ -9,12 +9,14 @@ export const Label: FC<{
   className?: string
   onClick?: (id: number) => void
   isInteractive?: boolean
+  disabled?: boolean
 }> = ({
   label,
   onClick = () => undefined,
   isActive,
   className = '',
   isInteractive = true,
+  disabled = false,
 }) => {
   const Icon = icons[label.fields.icon as keyof typeof icons] || Fragment
   return (
@@ -24,20 +26,24 @@ export const Label: FC<{
         className={classNames(
           className,
           `py-1.5 text-lg flex gap-2 text-left leading-6 pl-2 pr-3 rounded`,
+          `transition motion-reduce:transition-none`,
           isActive &&
-            isInteractive &&
+            isActive &&
+            !disabled &&
             `bg-primary border-primary text-white hover:bg-purple-400`,
-          (!isActive || !isInteractive) && ` border-gray-20`,
+          (!isInteractive || disabled) && `ring-1 ring-inset ring-gray-20`,
           !isActive &&
+            !disabled &&
             isInteractive &&
             'bg-purple-50 hover:bg-purple-200 text-purple-700',
-          isInteractive && [
-            `focus:outline-none focus:ring-2 focus:ring-primary`,
-            `focus:ring-offset-2 focus:ring-offset-white`,
-          ],
+          isInteractive &&
+            !disabled && [
+              `focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-primary`,
+              `focus-visible:ring-offset-2 focus-visible:ring-offset-white`,
+            ],
           !isInteractive && `cursor-default`
         )}
-        disabled={!isInteractive}
+        disabled={!isInteractive || disabled}
       >
         {label.fields.icon && (
           <Icon
diff --git a/src/components/Listbox.tsx b/src/components/Listbox.tsx
index db571254..2de2bd2d 100644
--- a/src/components/Listbox.tsx
+++ b/src/components/Listbox.tsx
@@ -16,6 +16,7 @@ interface ListboxType {
   onChange?: (selectedValue: number | null) => void
   nullSelectionLabel?: string
   className?: string
+  disabled?: boolean
 }
 
 export const Listbox: FC<ListboxType> = ({
@@ -25,18 +26,23 @@ export const Listbox: FC<ListboxType> = ({
   onChange = () => undefined,
   nullSelectionLabel = 'Keine Präferenz',
   className = '',
+  disabled = false,
 }) => {
   const handleChange = (selectedOption: number): void =>
     onChange(selectedOption)
 
-  const mergedOptions = [{ label: nullSelectionLabel, value: null }, ...options]
-  const selectedOption = mergedOptions.find(
-    (option) => option.value === activeValue
-  )
+  const nullSection = { label: nullSelectionLabel, value: null }
+  const mergedOptions = [nullSection, ...options]
+  const selectedOption =
+    mergedOptions.find((option) => option.value === activeValue) || nullSection
 
   return (
     <div className={classNames(className, 'w-full')}>
-      <HeadlessListbox value={activeValue} onChange={handleChange}>
+      <HeadlessListbox
+        value={activeValue}
+        onChange={handleChange}
+        disabled={disabled}
+      >
         <HeadlessListbox.Label
           className={classNames('w-full flex justify-between', 'text-lg')}
         >
diff --git a/src/components/TextSearch.tsx b/src/components/TextSearch.tsx
index 2cdd485f..a8843e70 100644
--- a/src/components/TextSearch.tsx
+++ b/src/components/TextSearch.tsx
@@ -17,6 +17,7 @@ interface StateType {
 
 interface TextSearchProps extends StateType {
   onChange: (state: Partial<StateType>) => void
+  disabled?: boolean
 }
 
 type CategoriesTextMapType = Record<keyof CategoriesType, string>
@@ -25,6 +26,7 @@ function TextSearch({
   text: initialText,
   categories,
   onChange,
+  disabled = false,
 }: TextSearchProps): JSX.Element {
   const texts = useTexts()
   const [text, setText] = useState(initialText || '')
@@ -43,6 +45,7 @@ function TextSearch({
     <fieldset
       className="w-full @md:w-[324px]"
       aria-labelledby="textSearchLabel"
+      disabled={disabled}
     >
       <TextInput
         id="textSearch"
@@ -58,12 +61,14 @@ function TextSearch({
           }
         }}
         value={text}
+        disabled={disabled}
       />
       {checkboxes.map(({ id, labelText }) => (
         <Checkbox
           key={id}
           id={id}
           labelText={labelText}
+          disabled={disabled}
           onChange={(evt) => {
             onChange({
               text,
diff --git a/src/components/icons/Spinner.tsx b/src/components/icons/Spinner.tsx
new file mode 100644
index 00000000..d21deed2
--- /dev/null
+++ b/src/components/icons/Spinner.tsx
@@ -0,0 +1,27 @@
+import { FC, SVGAttributes } from 'react'
+
+export const Spinner: FC<{
+  className?: SVGAttributes<SVGSVGElement>['className']
+}> = ({ className = '' }) => (
+  <svg
+    xmlns="http://www.w3.org/2000/svg"
+    width="24"
+    height="24"
+    viewBox="0 0 24 24"
+    fill="none"
+    stroke="currentColor"
+    strokeWidth="2"
+    strokeLinecap="round"
+    strokeLinejoin="round"
+    className={className}
+  >
+    <line x1="12" y1="2" x2="12" y2="6"></line>
+    <line x1="12" y1="18" x2="12" y2="22"></line>
+    <line x1="4.93" y1="4.93" x2="7.76" y2="7.76"></line>
+    <line x1="16.24" y1="16.24" x2="19.07" y2="19.07"></line>
+    <line x1="2" y1="12" x2="6" y2="12"></line>
+    <line x1="18" y1="12" x2="22" y2="12"></line>
+    <line x1="4.93" y1="19.07" x2="7.76" y2="16.24"></line>
+    <line x1="16.24" y1="7.76" x2="19.07" y2="4.93"></line>
+  </svg>
+)
diff --git a/src/lib/UrlStateContext.tsx b/src/lib/UrlStateContext.tsx
index 65cddcc5..4eea5b1a 100644
--- a/src/lib/UrlStateContext.tsx
+++ b/src/lib/UrlStateContext.tsx
@@ -48,8 +48,17 @@ export const UrlStateProvider: FC = ({ children }) => {
     if (typeof window === 'undefined') return
     const urlParams = new URLSearchParams(window.location.search)
     if (urlParams.get('qCategories')) return
+    const parsedQuery = mapRawQueryToState({
+      q: urlParams.get('q') || undefined,
+      qCategories: urlParams.getAll('qCategories') || undefined,
+      tags: urlParams.getAll('tags') || undefined,
+      back: urlParams.get('back') || undefined,
+      latitude: urlParams.get('latitude') || undefined,
+      longitude: urlParams.get('longitude') || undefined,
+      zoom: urlParams.get('zoom') || undefined,
+    })
     updateUrlState({
-      ...query,
+      ...parsedQuery,
       qCategories: stateSearchCategoriesToUrlSearchCategories({
         categorySelfHelp: true,
         categoryAdvising: true,