Test with Svelte 5

[dimfeld]

This task covers testing that svelte-maplibre works with Svelte 5 as is, and any fixes to be made while keeping the project compatible with Svelte 4.

[ivyrze]

When using the library with Svelte 5 and I'm getting a bunch of new warnings:

[vite-plugin-svelte] node_modules/svelte-maplibre/dist/AttributionControl.svelte:11:18 Properties of objects and arrays are not reactive unless in runes mode. Changes to this property will not cause the reactive statement to update
 9: $:
10:   if ($map && !control) {
11:     control = new maplibregl.AttributionControl({
                                                   ^
12:       compact,
13:       customAttribution
[vite-plugin-svelte] node_modules/svelte-maplibre/dist/FullscreenControl.svelte:16:18 Properties of objects and arrays are not reactive unless in runes mode. Changes to this property will not cause the reactive statement to update
14:       containerEl = container;
15:     }
16:     control = new maplibregl.FullscreenControl({
                                                  ^
17:       container: containerEl
18:     });
[vite-plugin-svelte] node_modules/svelte-maplibre/dist/GeolocateControl.svelte:14:18 Properties of objects and arrays are not reactive unless in runes mode. Changes to this property will not cause the reactive statement to update
12: $:
13:   if ($map && !control) {
14:     control = new maplibregl.GeolocateControl({
                                                 ^
15:       positionOptions,
16:       fitBoundsOptions,

...and so forth for the rest of the controls. Turning on runes mode breaks the build altogether, of course, because the package is still using the Svelte 4 syntax.

[benmccann]

I've sent a PR to Svelte to remove that warning: sveltejs/svelte#14111

What do you think about upgrading this project to use runes and requiring Svelte 5?

[dimfeld]

Definitely want to do it at some point. I’m busy with a new startup right now (using svelte 5 😁) but hope to have some time later this month for that.

…

On Sat Nov 2, 2024, 05:52 PM GMT, Ben McCann ***@***.***> wrote: I've sent a PR to Svelte to remove that warning: sveltejs/svelte#14111 <sveltejs/svelte#14111> What do you think about upgrading this project to use runes and requiring Svelte 5? — Reply to this email directly, view it on GitHub <#196 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAKOLVWMPULJ5E5SRPDJM4DZ6UGNXAVCNFSM6AAAAABMYQNPJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGA3DCNZYGI>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[joleeee]

I'm having trouble even just doing

    <MapLibre
        bind:map

on svelte 5. map stays undefined until after onMount is ran

(not even in runes mode)