From c531f02dc85ddeeab499af4bcdbc426cfcbaa759 Mon Sep 17 00:00:00 2001 From: Paul Weerheim Date: Fri, 4 Sep 2026 15:09:24 +0200 Subject: [PATCH 1/3] Fix transparent dialog panel and cap modal size The theme mapped every colour token except --color-background, so the bg-background class of Dialog.Content produced nothing and the panel stayed transparent. Both containers also had no height cap, so a tall modal grew past the window and took its footer with it. They now stop at 90% of the window in both directions, and the content scrolls. Modal registered its Escape listener once, and only when canCloseModal was true at that moment. A later change never reached it. --- src/lib/components/modals/Modal.svelte | 24 ++++++++++++------- .../ui/dialog/dialog-content.svelte | 4 +++- src/tailwind.css | 2 ++ 3 files changed, 21 insertions(+), 9 deletions(-) diff --git a/src/lib/components/modals/Modal.svelte b/src/lib/components/modals/Modal.svelte index 32c7ca2..fd1692f 100644 --- a/src/lib/components/modals/Modal.svelte +++ b/src/lib/components/modals/Modal.svelte @@ -11,19 +11,17 @@ let shortRandomString = Utils.uuidv4().slice(0, 8); function closeModalOnEscape(event: KeyboardEvent) { - if (event.key === 'Escape') { + // The test reads the value of now. A caller can block the close while a + // task runs, and release it after. + if (event.key === 'Escape' && canCloseModal) { onClose(); } } onMount(() => { - if (canCloseModal) { - //also add escape key listener to close modal - document.addEventListener('keydown', closeModalOnEscape); - } + document.addEventListener('keydown', closeModalOnEscape); return () => { - // Cleanup: remove event listener if it was added document.removeEventListener('keydown', closeModalOnEscape); }; }); @@ -53,7 +51,7 @@ {/if} -
+
@@ -78,10 +76,20 @@ background: #fff; padding: 1rem; border-radius: 0.5rem; + // A modal never takes more than 90% of the window, in both directions. + // The header and the footer keep their place, and the content scrolls. + display: flex; + flex-direction: column; width: 90%; - max-width: var(--width, 400px); + max-width: min(var(--width, 400px), 90vw); + max-height: 90vh; box-shadow: 0 2px 10px rgba(0, 0, 0, 0.1); + .modal-content { + min-height: 0; + overflow-y: auto; + } + header { font-size: 1.25rem; margin-bottom: 1rem; diff --git a/src/lib/components/ui/dialog/dialog-content.svelte b/src/lib/components/ui/dialog/dialog-content.svelte index 99cc73b..4d3e0e2 100644 --- a/src/lib/components/ui/dialog/dialog-content.svelte +++ b/src/lib/components/ui/dialog/dialog-content.svelte @@ -25,7 +25,9 @@ bind:ref data-slot="dialog-content" class={cn( - "bg-background data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 fixed left-[50%] top-[50%] z-50 grid w-full max-w-[calc(100%-2rem)] translate-x-[-50%] translate-y-[-50%] gap-4 rounded-lg border p-6 shadow-lg duration-200 sm:max-w-lg", + // A dialog never takes more than 90% of the window, in both directions. + // Content past that height scrolls, so the footer stays reachable. + "bg-background data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 fixed left-[50%] top-[50%] z-50 grid max-h-[90vh] w-full max-w-[90vw] translate-x-[-50%] translate-y-[-50%] gap-4 overflow-y-auto rounded-lg border p-6 shadow-lg duration-200 sm:max-w-lg", className )} {...restProps} diff --git a/src/tailwind.css b/src/tailwind.css index 487cff4..5a8fc5c 100644 --- a/src/tailwind.css +++ b/src/tailwind.css @@ -105,6 +105,8 @@ --radius-md: calc(var(--radius) - 2px); --radius-lg: var(--radius); --radius-xl: calc(var(--radius) + 4px); + /* `bg-background` needs this map. The dialog panel is transparent without it. */ + --color-background: var(--background); --color-foreground: var(--foreground); --color-card: var(--card); --color-card-foreground: var(--card-foreground); From 4b5861421beb4bc1fd10bef262223596e2741f3c Mon Sep 17 00:00:00 2001 From: Paul Weerheim Date: Fri, 4 Sep 2026 15:09:31 +0200 Subject: [PATCH 2/3] Add geospatial filter modal to query builder A query could only get an area filter after a run on the map viewer. The modal draws one in the builder, before the first run. The area now carries the two columns it tests. The compile resolved that pair by name detection alone, so a table with x/y columns could not take an area. The user picks the pair in the modal, and detection stays the fallback. --- AGENTS.md | 3 +- .../GeospatialFilterModal.svelte | 318 ++++++++++++++++++ .../QueryBuilderParameterBlock.svelte | 196 ++++++++--- .../query-builder/QueryWorkspace.svelte.ts | 29 +- .../visualisation/MapDrawTools.svelte | 32 +- .../visualisation/SpatialFilterMap.svelte | 95 ++++++ src/lib/geo/spatial-selection.ts | 79 ++++- src/lib/query/draft.ts | 15 +- src/lib/query/seed-hydration.ts | 8 +- .../visualisations/map-viewer/+page.svelte | 24 +- 10 files changed, 715 insertions(+), 84 deletions(-) create mode 100644 src/lib/components/query-builder/GeospatialFilterModal.svelte create mode 100644 src/lib/components/visualisation/SpatialFilterMap.svelte diff --git a/AGENTS.md b/AGENTS.md index 7dc6090..6b0b8e9 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -104,6 +104,7 @@ This file is a quick operational guide for coding agents working in this reposit - Polygon, box and cross section all end as one closed ring, so there is one filter kind. A cross section is a line plus a width; `crossSectionRing` in `geo/spatial-selection.ts` converts it. - Always send the bounding box of the polygon beside it, as two `MinMaxFilter`s on the latitude and longitude columns. The server can prune data with those, but not with the polygon test. `compileDraft` derives the box; never store it on a field. - `QueryDraft.spatialFilter` holds the area, because it applies to two columns and has no card of its own. `QueryWorkspace.updateActiveSpatialFilter` writes it, and also handles a block that has no draft (share link, JSON editor) by patching `compiled.filters`. +- The area carries the two columns it tests (`latitudeColumn` / `longitudeColumn`), because the user can pick another pair than the names say, for example `x` and `y`. Resolve the pair with `selectionColumns` (`geo/spatial-selection.ts`), never with `detectCoordinateColumns` alone: the names on the area win while the query still selects them, and detection is only the fallback for an older record. The draw tools rebuild the area on every shape change and drop the two names, so stamp them back with `withColumns` at the point of apply. - Terra Draw (`terra-draw` + `terra-draw-maplibre-gl-adapter`) draws the shape. After a shape is complete `MapDrawTools.svelte` clears Terra Draw and renders the ring in its own MapLibre source, so a loaded area and a new area look the same. ## Layer Rule (Important) @@ -170,7 +171,7 @@ Imports point one way only: ## Known Repo Facts - Static adapter outputs to `build/` and uses `fallback: 'index.html'`. -- Monaco editor and Perspective viewer are included and loaded client-side. +- Monaco editor is included and loaded client-side. - Project currently contains generated `build/` artifacts in repo; avoid editing generated files directly unless explicitly asked. ## Comment Rules diff --git a/src/lib/components/query-builder/GeospatialFilterModal.svelte b/src/lib/components/query-builder/GeospatialFilterModal.svelte new file mode 100644 index 0000000..97c17f7 --- /dev/null +++ b/src/lib/components/query-builder/GeospatialFilterModal.svelte @@ -0,0 +1,318 @@ + + +{#if open} + + (open = false)} + > +
+

+ Draw an area, and pick the two columns that the filter tests. +

+ + {#if candidates.length < 2} + + {:else} +
+
+ + + + + {latitudeColumn || 'Select a column'} + + + + Query columns + {#each candidates as column (column)} + {column} + {/each} + + + +
+ +
+ + + + + {longitudeColumn || 'Select a column'} + + + + Query columns + {#each candidates as column (column)} + {column} + {/each} + + + +
+
+ + {#if sameColumn} + + {/if} + {/if} + + +
+
+ (isDrawing = drawing)} + /> +
+ + +
+
+ +
+ {applyReason ?? ''} + + {#if selection} + + {/if} + + + + +
+ +{/if} + + diff --git a/src/lib/components/query-builder/QueryBuilderParameterBlock.svelte b/src/lib/components/query-builder/QueryBuilderParameterBlock.svelte index 7d79550..18012f2 100644 --- a/src/lib/components/query-builder/QueryBuilderParameterBlock.svelte +++ b/src/lib/components/query-builder/QueryBuilderParameterBlock.svelte @@ -3,8 +3,7 @@ * QueryBuilderParametrBlock.svelte * Author: Jasper van der Barg * Description: Query Builder Parameter Block Component - */ - + */ import * as Dialog from '$lib/components/ui/dialog/index.js'; import * as SearchSelect from '$lib/components/ui/search-select/index.js'; @@ -12,23 +11,28 @@ import CheckIcon from '@lucide/svelte/icons/check'; import PlusIcon from '@lucide/svelte/icons/plus'; import { BeaconClient } from '@/beacon-api/client'; - import type { CompiledQuery, DataType, OutputFormat } from '@/beacon-api/types'; + import type { CompiledQuery, DataType } from '@/beacon-api/types'; import { Utils } from '@/utils'; import Parameter from './Parameter.svelte'; import type { SelectedFilterType } from '@/query/filter-types'; - import { QueryBuilder } from '@/beacon-api/query'; import { addToast } from '@/stores/toasts'; import type { QuerySelectionStatus } from '@/query/selection-status'; - import type { QueryActions } from './QueryActions'; + import type { QueryActions } from './QueryActions'; import { compileDraft, defaultOutputFormat, type QueryDraft } from '@/query/draft'; import MapPinnedIcon from '@lucide/svelte/icons/map-pinned'; import TriangleAlertIcon from '@lucide/svelte/icons/triangle-alert'; import XIcon from '@lucide/svelte/icons/x'; import { runBlockReason } from '@/query/query-guard'; import { settings } from '@/stores/settings'; - import { describeSelection, type SpatialSelection } from '@/geo/spatial-selection'; + import { + describeSelection, + missingSelectionColumn, + selectionColumns, + type SpatialSelection + } from '@/geo/spatial-selection'; import { hydrateDraftFromQuery } from '@/query/seed-hydration'; - + import GeospatialFilterModal from './GeospatialFilterModal.svelte'; + import PencilLineIcon from '@lucide/svelte/icons/pencil-line'; let { table_name, @@ -44,15 +48,15 @@ dataTable: '', columns: 0, filters: 0, - selection: 0, + selection: 0 }), - // Only the query actions live here. The workbench owns navigation, because - // it holds the StoredQuery block id for visualisation links. - actions = $bindable({ + // Only the query actions live here. The workbench owns navigation, because + // it holds the StoredQuery block id for visualisation links. + actions = $bindable({ compileQuery: compileQuery, - downloadData: handleSubmit, - resetQuery: undefined - }), + downloadData: handleSubmit, + resetQuery: undefined + }) }: { table_name: string; client: BeaconClient; @@ -68,7 +72,7 @@ /** Bound to the parent, which shows the output format control. */ selected_output_format?: string; status?: QuerySelectionStatus; - actions?: QueryActions; // todo + actions?: QueryActions; // todo } = $props(); let searchInput; @@ -82,11 +86,13 @@ let selectedFields: { name: string; type: DataType; selected_filters: SelectedFilterType[] }[] = $state(initialDraft ? Utils.cloneObject(initialDraft.selectedFields) : []); /** - * The area drawn on the map viewer. The builder shows it, and can remove it, - * but it cannot draw one. It applies to the latitude and the longitude column + * The area of this query. The user draws it in {@link GeospatialFilterModal}, + * or on the map viewer. It applies to the latitude and the longitude column * together, so it has no card of its own. */ let spatialFilter: SpatialSelection | null = $state(initialDraft?.spatialFilter ?? null); + /** True while the geospatial filter modal is open. */ + let isGeoFilterOpen = $state(false); /** Table the current selection belongs to; used to reset on a real table change. */ let selectionTable = initialDraft?.tableName ?? null; let hasInitialisedTable = $state(false); @@ -116,7 +122,44 @@ ); }); - const firstVisibleItem = $derived(fields.find((item) => !(item.ref as {hidden: boolean})?.hidden)); + /** The number columns of the query. An area filter can only test these. */ + const numericColumnCount = $derived( + selectedFields.filter((field) => Utils.isNumericDataType(field.type)).length + ); + + /** The reason that the query takes no area, or null. */ + const areaBlockReason = $derived.by(() => { + if (numericColumnCount < 2) { + return 'Select a latitude and a longitude column first.'; + } + + return null; + }); + + /** + * The column of the area that the query no longer selects, or null. + * + * The compile drops such a filter, so the user must know. The area stays on + * the draft, and the modal offers the pickers again. + */ + const missingAreaColumn = $derived( + missingSelectionColumn( + spatialFilter, + selectedFields.map((field) => field.name) + ) + ); + + /** The two columns the area tests. The compile resolves the same pair. */ + const areaColumns = $derived( + selectionColumns( + spatialFilter, + selectedFields.map((field) => field.name) + ) + ); + + const firstVisibleItem = $derived( + fields.find((item) => !(item.ref as { hidden: boolean })?.hidden) + ); const firstVisibleMatchingItem = $derived.by(() => { const normalized = searchQuery.trim().toLowerCase(); @@ -125,9 +168,7 @@ return firstVisibleItem; } - return fields.find( - (item) => item.name.toLowerCase().includes(normalized) - ); + return fields.find((item) => item.name.toLowerCase().includes(normalized)); }); // $effect(() => { @@ -159,7 +200,6 @@ selectionTable = initialDraft.tableName; }); - // Load the schema for the selected table (cached per instance). Needed for the // "Add Parameter" list and to parse a deep-link seed. Does NOT clear the current // selection — that only happens on a real table change (below). @@ -219,6 +259,8 @@ if (table !== selectionTable) { selectedFields = []; + // The area names two columns of the old table, so it goes with them. + spatialFilter = null; } selectionTable = table; @@ -274,7 +316,7 @@ if (index !== -1) { const selectedIndex = selectedFields.findIndex((f) => f.name === field_name); - + if (selectedIndex === -1) { selectedFields.push({ name: fields[index].name, @@ -299,25 +341,25 @@ } // onReset = resetBuilder; - actions.resetQuery = resetBuilder; + actions.resetQuery = resetBuilder; + /** + * The query of the controls above. `compileDraft` is the one compile rule, so + * a download from here also carries the area filter. + */ function compileQuery(): CompiledQuery { - let builder = new QueryBuilder(); - - for (const field of selectedFields) { - builder.addSelect({ column: field.name, alias: null }); - for (const filter of field.selected_filters) { - let bfilter = Utils.parameterFilterTypeToFilter(filter.filter_value, field.name); - if (bfilter) { - builder.addFilter(bfilter); - } - } - } + const compiled = compileDraft({ + tableName: table_name, + selectedFields, + outputFormat: selected_output_format, + spatialFilter + }); - builder.setFrom(table_name); - builder.setOutput({ format: selected_output_format as OutputFormat }); + if (!compiled) { + throw new Error('Pick a table and at least one column.'); + } - return builder.compile(); + return compiled; } actions.compileQuery = compileQuery; @@ -383,12 +425,32 @@

Query Parameters

- +
+ + {#if !spatialFilter} + + {/if} +
+ (spatialFilter = next)} + /> + @@ -418,7 +480,8 @@ bind:this={field.ref} > {field.name} - {Utils.dataTypeToString(field.type)} + {Utils.dataTypeToString(field.type)} {#if selectedFields.find((f) => f.name === field.name)} {/if} @@ -441,12 +504,31 @@
Area: {describeSelection(spatialFilter)} - Drawn on the map viewer + + {#if missingAreaColumn} + + + The query does not select "{missingAreaColumn}", so this area filters nothing. + + {:else if areaColumns} + + {areaColumns.latitude} / {areaColumns.longitude} + + {:else} + + + The query selects no latitude and longitude column, so this area filters nothing. + + {/if} + +
@@ -461,11 +543,9 @@

No parameters selected, use the 'Add Parameter' button above.

{/if}
-
diff --git a/src/lib/geo/spatial-selection.ts b/src/lib/geo/spatial-selection.ts index 1d696e5..7ca72c1 100644 --- a/src/lib/geo/spatial-selection.ts +++ b/src/lib/geo/spatial-selection.ts @@ -10,6 +10,7 @@ * columns. See `beacon-core/src/query/filter/geo_json.rs`. */ import type { GeoJsonFilter, GeoJsonPolygon, MinMaxFilter } from '@/beacon-api/types'; +import { detectCoordinateColumns } from '@/geo/coordinate-columns'; import { getSettings } from '@/stores/settings'; export type SpatialSelectionMode = 'polygon' | 'box' | 'cross-section'; @@ -25,6 +26,19 @@ export type SpatialSelection = { line?: LngLat[]; /** Cross section only: the full width of the band, in kilometres. */ widthKm?: number; + /** + * The columns the filter tests. The user picks them in the query builder. + * Both are absent on an area of an older record, and {@link selectionColumns} + * then falls back to {@link detectCoordinateColumns}. + */ + latitudeColumn?: string; + longitudeColumn?: string; +}; + +/** The pair of columns a spatial filter tests. */ +export type CoordinatePair = { + latitude: string; + longitude: string; }; export type Bounds = { @@ -223,6 +237,67 @@ export function makeCrossSectionSelection(line: LngLat[], widthKm: number): Spat }; } +/** Put the two column names on a selection. The draw tools drop them. */ +export function withColumns( + selection: SpatialSelection, + columns: CoordinatePair | null +): SpatialSelection { + if (!columns) return selection; + + return { + ...selection, + latitudeColumn: columns.latitude, + longitudeColumn: columns.longitude + }; +} + +/** + * The two columns a spatial filter must test, or null. + * + * The names on the selection win, because the user picked them. They only win + * while the query still selects both: a filter on a column that the query does + * not select is invalid. Detection then answers, which also serves an area of + * an older record, and an area that the map viewer drew. + */ +export function selectionColumns( + selection: SpatialSelection | null | undefined, + availableNames: string[] +): CoordinatePair | null { + const latitude = selection?.latitudeColumn; + const longitude = selection?.longitudeColumn; + + if ( + latitude && + longitude && + availableNames.includes(latitude) && + availableNames.includes(longitude) + ) { + return { latitude, longitude }; + } + + const detection = detectCoordinateColumns(availableNames); + if (!detection.latitude || !detection.longitude) return null; + + return { latitude: detection.latitude.name, longitude: detection.longitude.name }; +} + +/** + * The column of a selection that the query does not select, or null. + * + * The builder reports this. The area stays on the draft, so the user can pick + * the column again, or select it in the query. + */ +export function missingSelectionColumn( + selection: SpatialSelection | null | undefined, + availableNames: string[] +): string | null { + const wanted = [selection?.latitudeColumn, selection?.longitudeColumn].filter( + Boolean + ) as string[]; + + return wanted.find((name) => !availableNames.includes(name)) ?? null; +} + /** True when the selection has a usable area. */ export function isUsableSelection(selection: SpatialSelection | null | undefined): boolean { return !!selection && selection.ring.length >= 4; @@ -284,7 +359,9 @@ export function fromGeoJsonFilter(filter: GeoJsonFilter): SpatialSelection | nul return { mode: 'polygon', - ring: ring.map((point) => [Number(point[0]), Number(point[1])] as LngLat) + ring: ring.map((point) => [Number(point[0]), Number(point[1])] as LngLat), + latitudeColumn: filter.latitude_query_parameter, + longitudeColumn: filter.longitude_query_parameter }; } diff --git a/src/lib/query/draft.ts b/src/lib/query/draft.ts index ab22b15..9f79200 100644 --- a/src/lib/query/draft.ts +++ b/src/lib/query/draft.ts @@ -15,11 +15,11 @@ import { getSettings } from '@/stores/settings'; import { Utils } from '@/utils'; import { isUsableSelection, + selectionColumns, toBboxFilters, toGeoJsonFilter, type SpatialSelection } from '@/geo/spatial-selection'; -import { detectCoordinateColumns } from '@/geo/coordinate-columns'; /** A single selected column plus the filters applied to it. */ export type SelectedField = { @@ -102,20 +102,21 @@ export function compileDraft(draft: QueryDraft | null | undefined): CompiledQuer * The box is always derived here, and is never stored on a field. So one delete * of `spatialFilter` removes every part of the area again. * - * The query keeps no filter when it does not select both a latitude and a - * longitude column. + * The two columns come from the area itself, because the user picks them in the + * builder. See {@link selectionColumns}. The query keeps no filter while it + * selects neither pair. */ function addSpatialFilters(builder: QueryBuilder, draft: QueryDraft): void { const selection = draft.spatialFilter; if (!isUsableSelection(selection)) return; const names = draft.selectedFields.map((field) => field.name); - const { latitude, longitude } = detectCoordinateColumns(names); - if (!latitude || !longitude) return; + const columns = selectionColumns(selection, names); + if (!columns) return; - builder.addFilter(toGeoJsonFilter(selection!, latitude.name, longitude.name)); + builder.addFilter(toGeoJsonFilter(selection!, columns.latitude, columns.longitude)); - for (const filter of toBboxFilters(selection!, latitude.name, longitude.name)) { + for (const filter of toBboxFilters(selection!, columns.latitude, columns.longitude)) { builder.addFilter(filter); } } diff --git a/src/lib/query/seed-hydration.ts b/src/lib/query/seed-hydration.ts index 8f11328..8e0c2f4 100644 --- a/src/lib/query/seed-hydration.ts +++ b/src/lib/query/seed-hydration.ts @@ -139,16 +139,18 @@ function isDerivedBoxFilter( return false; } - const name = filter.for_query_parameter.toLowerCase(); + const name = filter.for_query_parameter; const matches = (min: number, max: number) => { return Math.abs(Number(filter.min) - min) < 1e-9 && Math.abs(Number(filter.max) - max) < 1e-9; }; - if (name.includes('latitude')) { + // The geo filter names its own two columns, so an area on `x` and `y` also + // matches. A name test for "latitude" holds only for the default pair. + if (name === selection.latitudeColumn) { return matches(bounds.minLat, bounds.maxLat); } - if (name.includes('longitude')) { + if (name === selection.longitudeColumn) { return matches(bounds.minLon, bounds.maxLon); } diff --git a/src/routes/visualisations/map-viewer/+page.svelte b/src/routes/visualisations/map-viewer/+page.svelte index 975ec16..b51be76 100644 --- a/src/routes/visualisations/map-viewer/+page.svelte +++ b/src/routes/visualisations/map-viewer/+page.svelte @@ -16,7 +16,11 @@ import VisualisationTabs from '@/components/visualisation/VisualisationTabs.svelte'; import MapDrawTools from '@/components/visualisation/MapDrawTools.svelte'; import { MapViewController } from '@/components/visualisation/MapViewController.svelte'; - import type { SpatialSelection } from '@/geo/spatial-selection'; + import { + selectionColumns, + withColumns, + type SpatialSelection + } from '@/geo/spatial-selection'; import { addToast } from '@/stores/toasts'; import { runBlockReason } from '@/query/query-guard'; import { settings } from '@/stores/settings'; @@ -174,9 +178,23 @@ untrack(() => workspace.updateActiveMapView(state)); }); - /** Write the drawn area into the query. The effect above then re-runs it. */ + /** + * Write the drawn area into the query. The effect above then re-runs it. + * + * The area names the two columns it tests. The builder can pick another pair + * than the detection finds, for example `x` and `y`. A redraw here must keep + * that pair, so the columns come from the area of the block. + */ function applyAreaFilter() { - workspace.updateActiveSpatialFilter(selection); + if (!selection) { + workspace.updateActiveSpatialFilter(null); + return; + } + + const stored = workspace.activeBlock?.draft?.spatialFilter ?? selection; + const columns = selectionColumns(stored, map.availableColumnNames); + + workspace.updateActiveSpatialFilter(withColumns(selection, columns)); } From ff59224ec99dc573acd36e52664bb1050e619de1 Mon Sep 17 00:00:00 2001 From: Jasper-Maris <87017907+Jasper-Maris@users.noreply.github.com> Date: Fri, 4 Sep 2026 15:40:57 +0200 Subject: [PATCH 3/3] Fix indentation for MapDrawTools component Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- .../query-builder/GeospatialFilterModal.svelte | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/lib/components/query-builder/GeospatialFilterModal.svelte b/src/lib/components/query-builder/GeospatialFilterModal.svelte index 97c17f7..98895dc 100644 --- a/src/lib/components/query-builder/GeospatialFilterModal.svelte +++ b/src/lib/components/query-builder/GeospatialFilterModal.svelte @@ -204,11 +204,11 @@
(isDrawing = drawing)} - /> + {map} + bind:selection={draft} + showApply={false} + onDrawingChange={(drawing) => (isDrawing = drawing)} + />