From 12766964c05af407001135b36c59995191c662b7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jesper=20Engstr=C3=B6m?= <jesper.engstrom@kb.se>
Date: Wed, 11 Dec 2024 10:31:41 +0100
Subject: [PATCH] Accomplish url sync with $effect, not afterNavigate

---
 .../src/lib/components/CodeMirror.svelte       | 18 ++++++------------
 .../src/lib/components/SuperSearch.svelte      |  2 +-
 2 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/packages/supersearch/src/lib/components/CodeMirror.svelte b/packages/supersearch/src/lib/components/CodeMirror.svelte
index 93e06739e..5d15030e7 100644
--- a/packages/supersearch/src/lib/components/CodeMirror.svelte
+++ b/packages/supersearch/src/lib/components/CodeMirror.svelte
@@ -7,8 +7,7 @@
 
 <script lang="ts">
 	import { DEV } from 'esm-env';
-	import { onMount, tick } from 'svelte';
-	import { afterNavigate } from '$app/navigation';
+	import { onMount } from 'svelte';
 	import { EditorView } from '@codemirror/view';
 	import { EditorState, StateEffect, type Extension, type SelectionRange } from '@codemirror/state';
 	import isViewUpdateFromUserInput from '$lib/utils/isViewUpdateFromUserInput.js';
@@ -95,16 +94,11 @@
 		});
 	});
 
-	afterNavigate(() => {
-		tick().then(() => {
-			editorView?.dispatch({
-				changes: {
-					from: 0,
-					to: editorView.state.doc.length,
-					insert: value
-				}
-			});
-		});
+	$effect(() => {
+		if (value !== editorView?.state.doc.toString()) {
+			// Reset editor when value changes from outside (= user navigating)
+			reset({ doc: value });
+		}
 	});
 
 	$effect(() => {
diff --git a/packages/supersearch/src/lib/components/SuperSearch.svelte b/packages/supersearch/src/lib/components/SuperSearch.svelte
index 2d47f42d5..1a734354b 100644
--- a/packages/supersearch/src/lib/components/SuperSearch.svelte
+++ b/packages/supersearch/src/lib/components/SuperSearch.svelte
@@ -77,7 +77,7 @@
 	}
 
 	function handleChangeCodeMirror(event: ChangeCodeMirrorEvent) {
-		if (!dialog?.open) {
+		if (!dialog?.open && value !== event.value) {
 			showExpandedSearch();
 		}
 		value = event.value;