@genspectrum/dashboard-components 0.11.2 → 0.11.3

package/dist/style.css

@@ -2684,9 +2684,6 @@ input.tab:checked + .tab-content,
   border-end-end-radius: inherit;
   border-start-end-radius: inherit;
 }
-.modal-middle {
-  place-items: center;
-}
 .modal-bottom {
   place-items: end;
 }
@@ -3131,9 +3128,6 @@ input.tab:checked + .tab-content,
 .min-w-\[7\.5rem\] {
   min-width: 7.5rem;
 }
-.max-w-3xl {
-  max-width: 48rem;
-}
 .max-w-screen-lg {
   max-width: 1024px;
 }
@@ -3307,10 +3301,6 @@ input.tab:checked + .tab-content,
   padding-top: 0.5rem;
   padding-bottom: 0.5rem;
 }
-.py-4 {
-  padding-top: 1rem;
-  padding-bottom: 1rem;
-}
 .pl-2 {
   padding-left: 0.5rem;
 }

package/dist/util.d.ts

@@ -787,7 +787,10 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-mutation-comparison-component': MutationComparisonComponent;
+        'gs-location-filter': LocationFilterComponent;
+    }
+    interface HTMLElementEventMap {
+        'gs-location-changed': CustomEvent<Record<string, string>>;
     }
 }
 
@@ -795,7 +798,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-mutation-comparison-component': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-location-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -803,7 +806,11 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-mutations-component': MutationsComponent;
+        'gs-date-range-selector': DateRangeSelectorComponent;
+    }
+    interface HTMLElementEventMap {
+        'gs-date-range-filter-changed': CustomEvent<Record<string, string>>;
+        'gs-date-range-option-changed': DateRangeOptionChangedEvent;
     }
 }
 
@@ -811,7 +818,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-mutations-component': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-date-range-selector': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -819,7 +826,10 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-prevalence-over-time': PrevalenceOverTimeComponent;
+        'gs-text-input': TextInputComponent;
+    }
+    interface HTMLElementEventMap {
+        'gs-text-input-changed': CustomEvent<Record<string, string>>;
     }
 }
 
@@ -827,7 +837,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-prevalence-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-text-input': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -835,7 +845,10 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-relative-growth-advantage': RelativeGrowthAdvantageComponent;
+        'gs-mutation-filter': MutationFilterComponent;
+    }
+    interface HTMLElementEventMap {
+        'gs-mutation-filter-changed': CustomEvent<MutationsFilter>;
     }
 }
 
@@ -843,7 +856,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-relative-growth-advantage': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-mutation-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -851,7 +864,10 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-aggregate': AggregateComponent;
+        'gs-lineage-filter': LineageFilterComponent;
+    }
+    interface HTMLElementEventMap {
+        'gs-lineage-filter-changed': CustomEvent<Record<string, string>>;
     }
 }
 
@@ -859,7 +875,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-aggregate': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-lineage-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -867,7 +883,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-number-sequences-over-time': NumberSequencesOverTimeComponent;
+        'gs-mutation-comparison-component': MutationComparisonComponent;
     }
 }
 
@@ -875,7 +891,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-number-sequences-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-mutation-comparison-component': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -883,7 +899,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-mutations-over-time': MutationsOverTimeComponent;
+        'gs-mutations-component': MutationsComponent;
     }
 }
 
@@ -891,7 +907,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-mutations-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-mutations-component': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -899,7 +915,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-sequences-by-location': SequencesByLocationComponent;
+        'gs-prevalence-over-time': PrevalenceOverTimeComponent;
     }
 }
 
@@ -907,7 +923,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-sequences-by-location': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-prevalence-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -915,7 +931,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-statistics': StatisticsComponent;
+        'gs-relative-growth-advantage': RelativeGrowthAdvantageComponent;
     }
 }
 
@@ -923,7 +939,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-statistics': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-relative-growth-advantage': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -931,11 +947,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-date-range-selector': DateRangeSelectorComponent;
-    }
-    interface HTMLElementEventMap {
-        'gs-date-range-filter-changed': CustomEvent<Record<string, string>>;
-        'gs-date-range-option-changed': DateRangeOptionChangedEvent;
+        'gs-aggregate': AggregateComponent;
     }
 }
 
@@ -943,7 +955,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-date-range-selector': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-aggregate': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -951,10 +963,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-location-filter': LocationFilterComponent;
-    }
-    interface HTMLElementEventMap {
-        'gs-location-changed': CustomEvent<Record<string, string>>;
+        'gs-number-sequences-over-time': NumberSequencesOverTimeComponent;
     }
 }
 
@@ -962,7 +971,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-location-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-number-sequences-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -970,10 +979,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-text-input': TextInputComponent;
-    }
-    interface HTMLElementEventMap {
-        'gs-text-input-changed': CustomEvent<Record<string, string>>;
+        'gs-mutations-over-time': MutationsOverTimeComponent;
     }
 }
 
@@ -981,7 +987,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-text-input': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-mutations-over-time': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -989,10 +995,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-mutation-filter': MutationFilterComponent;
-    }
-    interface HTMLElementEventMap {
-        'gs-mutation-filter-changed': CustomEvent<MutationsFilter>;
+        'gs-sequences-by-location': SequencesByLocationComponent;
     }
 }
 
@@ -1000,7 +1003,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-mutation-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-sequences-by-location': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }
@@ -1008,10 +1011,7 @@ declare global {
 
 declare global {
     interface HTMLElementTagNameMap {
-        'gs-lineage-filter': LineageFilterComponent;
-    }
-    interface HTMLElementEventMap {
-        'gs-lineage-filter-changed': CustomEvent<Record<string, string>>;
+        'gs-statistics': StatisticsComponent;
     }
 }
 
@@ -1019,7 +1019,7 @@ declare global {
 declare global {
     namespace JSX {
         interface IntrinsicElements {
-            'gs-lineage-filter': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
+            'gs-statistics': DetailedHTMLProps<HTMLAttributes<HTMLElement>, HTMLElement>;
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@genspectrum/dashboard-components",
-    "version": "0.11.2",
+    "version": "0.11.3",
     "description": "GenSpectrum web components for building dashboards",
     "type": "module",
     "license": "AGPL-3.0-only",

package/src/preact/components/error-display.tsx

@@ -2,6 +2,8 @@ import { type FunctionComponent } from 'preact';
 import { useEffect, useRef } from 'preact/hooks';
 import { type ZodError } from 'zod';
 
+import { InfoHeadline1, InfoParagraph } from './info';
+import { Modal, useModalRef } from './modal';
 import { LapisError, UnknownLapisError } from '../../lapisApi/lapisApi';
 
 export const GS_ERROR_EVENT_TYPE = 'gs-error';
@@ -46,7 +48,7 @@ export const ErrorDisplay: FunctionComponent<ErrorDisplayProps> = ({ error, rese
     console.error(error);
 
     const containerRef = useRef<HTMLInputElement>(null);
-    const ref = useRef<HTMLDialogElement>(null);
+    const modalRef = useModalRef();
 
     useEffect(() => {
         containerRef.current?.dispatchEvent(new ErrorEvent(error));
@@ -66,23 +68,16 @@ export const ErrorDisplay: FunctionComponent<ErrorDisplayProps> = ({ error, rese
                     {details !== undefined && (
                         <>
                             {' '}
-                            <button className='underline hover:text-gray-400' onClick={() => ref.current?.showModal()}>
+                            <button
+                                className='underline hover:text-gray-400'
+                                onClick={() => modalRef.current?.showModal()}
+                            >
                                 Show details.
                             </button>
-                            <dialog ref={ref} class='modal'>
-                                <div class='modal-box'>
-                                    <form method='dialog'>
-                                        <button className='btn btn-sm btn-circle btn-ghost absolute right-2 top-2'>
-                                            ✕
-                                        </button>
-                                    </form>
-                                    <h1 class='text-lg'>{details.headline}</h1>
-                                    <div class='py-4'>{details.message}</div>
-                                </div>
-                                <form method='dialog' class='modal-backdrop'>
-                                    <button>close</button>
-                                </form>
-                            </dialog>
+                            <Modal modalRef={modalRef}>
+                                <InfoHeadline1>{details.headline}</InfoHeadline1>
+                                <InfoParagraph>{details.message}</InfoParagraph>
+                            </Modal>
                         </>
                     )}
                 </div>

package/src/preact/components/info.tsx

@@ -1,11 +1,12 @@
 import { type FunctionComponent } from 'preact';
-import { useRef } from 'preact/hooks';
+
+import { Modal, useModalRef } from './modal';
 
 const Info: FunctionComponent = ({ children }) => {
-    const dialogRef = useRef<HTMLDialogElement>(null);
+    const modalRef = useModalRef();
 
     const toggleHelp = () => {
-        dialogRef.current?.showModal();
+        modalRef.current?.showModal();
     };
 
     return (
@@ -13,22 +14,7 @@ const Info: FunctionComponent = ({ children }) => {
             <button type='button' className='btn btn-xs' onClick={toggleHelp}>
                 ?
             </button>
-            <dialog ref={dialogRef} className={'modal modal-bottom sm:modal-middle'}>
-                <div className='modal-box sm:max-w-5xl'>
-                    <form method='dialog'>
-                        <button className='btn btn-sm btn-circle btn-ghost absolute right-2 top-2'>✕</button>
-                    </form>
-                    <div className={'flex flex-col'}>{children}</div>
-                    <div className='modal-action'>
-                        <form method='dialog'>
-                            <button className={'float-right underline text-sm hover:text-blue-700 mr-2'}>Close</button>
-                        </form>
-                    </div>
-                </div>
-                <form method='dialog' className='modal-backdrop'>
-                    <button>Helper to close when clicked outside</button>
-                </form>
-            </dialog>
+            <Modal modalRef={modalRef}>{children}</Modal>
         </div>
     );
 };

package/src/preact/components/modal.stories.tsx

@@ -0,0 +1,44 @@
+import { type Meta, type StoryObj } from '@storybook/preact';
+import { expect, waitFor, within } from '@storybook/test';
+import { type FunctionComponent } from 'preact';
+
+import { Modal, type ModalProps, useModalRef } from './modal';
+
+const meta: Meta<ModalProps> = {
+    title: 'Component/Modal',
+    component: Modal,
+    parameters: { fetchMock: {} },
+};
+
+export default meta;
+
+const WrapperWithButtonThatOpensTheModal: FunctionComponent = () => {
+    const modalRef = useModalRef();
+
+    return (
+        <div>
+            <button className='btn' onClick={() => modalRef.current?.showModal()}>
+                Open modal
+            </button>
+            <Modal modalRef={modalRef}>
+                <h1>Modal content</h1>
+            </Modal>
+        </div>
+    );
+};
+
+export const ModalStory: StoryObj<ModalProps> = {
+    render: () => {
+        return <WrapperWithButtonThatOpensTheModal />;
+    },
+    play: async ({ canvasElement, step }) => {
+        const canvas = within(canvasElement);
+
+        await step('Open the modal', async () => {
+            const button = canvas.getByText('Open modal');
+            button.click();
+
+            await waitFor(() => expect(canvas.getByText('Modal content')).toBeVisible());
+        });
+    },
+};

package/src/preact/components/modal.tsx

@@ -0,0 +1,31 @@
+import { type FunctionComponent, type Ref } from 'preact';
+import { useRef } from 'preact/hooks';
+
+export type ModalProps = {
+    modalRef: Ref<HTMLDialogElement>;
+};
+
+export function useModalRef() {
+    return useRef<HTMLDialogElement>(null);
+}
+
+export const Modal: FunctionComponent<ModalProps> = ({ children, modalRef }) => {
+    return (
+        <dialog ref={modalRef} className={'modal modal-bottom sm:modal-middle'}>
+            <div className='modal-box sm:max-w-5xl'>
+                <form method='dialog'>
+                    <button className='btn btn-sm btn-circle btn-ghost absolute right-2 top-2'>✕</button>
+                </form>
+                <div className={'flex flex-col'}>{children}</div>
+                <div className='modal-action'>
+                    <form method='dialog'>
+                        <button className={'float-right underline text-sm hover:text-blue-700 mr-2'}>Close</button>
+                    </form>
+                </div>
+            </div>
+            <form method='dialog' className='modal-backdrop'>
+                <button>Helper to close when clicked outside</button>
+            </form>
+        </dialog>
+    );
+};

package/src/preact/map/sequences-by-location-map.tsx

@@ -7,6 +7,7 @@ import { type GeoJsonFeatureProperties, type MapSource, useGeoJsonMap } from './
 import { type AggregateData } from '../../query/queryAggregateData';
 import { InfoHeadline1, InfoParagraph } from '../components/info';
 import { LoadingDisplay } from '../components/loading-display';
+import { Modal, useModalRef } from '../components/modal';
 import { formatProportion } from '../shared/table/formatProportion';
 
 type FeatureData = { proportion: number; count: number };
@@ -90,7 +91,7 @@ export const SequencesByLocationMapInner: FunctionComponent<SequencesByLocationM
             style: (feature: Feature<GeometryObject, EnhancedGeoJsonFeatureProperties> | undefined) => ({
                 fillColor: getColor(feature?.properties.data?.proportion),
                 fillOpacity: 1,
-                color: 'grey',
+                color: '#666666',
                 weight: 1,
             }),
         })
@@ -135,48 +136,38 @@ const DataMatchInformation: FunctionComponent<DataMatchInformationProps> = ({
     nullCount,
     hasTableView,
 }) => {
-    const dialogRef = useRef<HTMLDialogElement>(null);
+    const modalRef = useModalRef();
 
     const proportion = formatProportion(countOfMatchedLocationData / totalCount);
 
     return (
         <>
             <button
-                onClick={() => dialogRef.current?.showModal()}
+                onClick={() => modalRef.current?.showModal()}
                 className='text-sm absolute bottom-0 px-1 z-[1001] bg-white rounded border cursor-pointer tooltip'
                 data-tip='Click for detailed information'
             >
                 This map shows {proportion} of the data.
             </button>
-            <dialog ref={dialogRef} className={'modal modal-middle'}>
-                <div className='modal-box max-w-3xl'>
-                    <InfoHeadline1>Sequences By Location - Map View</InfoHeadline1>
-                    <InfoParagraph>
-                        The current filter has matched {totalCount.toLocaleString('en-us')} sequences. From these
-                        sequences, we were able to match {countOfMatchedLocationData.toLocaleString('en-us')} (
-                        {proportion}) on locations on the map.
-                    </InfoParagraph>
-                    <InfoParagraph>
-                        {unmatchedLocations.length > 0 && (
-                            <>
-                                The following locations from the data could not be matched on the map:{' '}
-                                {unmatchedLocations.map((it) => `"${it}"`).join(', ')}.{' '}
-                            </>
-                        )}
-                        {nullCount > 0 &&
-                            `${nullCount.toLocaleString('en-us')} matching sequences have no location information. `}
-                        {hasTableView && 'You can check the table view for more detailed information.'}
-                    </InfoParagraph>
-                    <div className='modal-action'>
-                        <form method='dialog'>
-                            <button className={'float-right underline text-sm hover:text-blue-700 mr-2'}>Close</button>
-                        </form>
-                    </div>
-                </div>
-                <form method='dialog' className='modal-backdrop'>
-                    <button>Helper to close when clicked outside</button>
-                </form>
-            </dialog>
+            <Modal modalRef={modalRef}>
+                <InfoHeadline1>Sequences By Location - Map View</InfoHeadline1>
+                <InfoParagraph>
+                    The current filter has matched {totalCount.toLocaleString('en-us')} sequences. From these sequences,
+                    we were able to match {countOfMatchedLocationData.toLocaleString('en-us')} ({proportion}) on
+                    locations on the map.
+                </InfoParagraph>
+                <InfoParagraph>
+                    {unmatchedLocations.length > 0 && (
+                        <>
+                            The following locations from the data could not be matched on the map:{' '}
+                            {unmatchedLocations.map((it) => `"${it}"`).join(', ')}.{' '}
+                        </>
+                    )}
+                    {nullCount > 0 &&
+                        `${nullCount.toLocaleString('en-us')} matching sequences have no location information. `}
+                    {hasTableView && 'You can check the table view for more detailed information.'}
+                </InfoParagraph>
+            </Modal>
         </>
     );
 };
@@ -232,7 +223,7 @@ function matchLocationDataAndGeoJsonFeatures(
 
 function getColor(value: number | undefined): string {
     if (value === undefined) {
-        return '#888888';
+        return '#DDDDDD';
     }
 
     const thresholds = [