fix-svelte-check-errors-and-warnings.md 1.1 KB

Fix svelte-check errors and warnings

Branch: svelte-check-fixes → main

What changed

bajdev's npm run check on the M4 code reported 4 errors and 3 warnings. This fixes all of them. (The fix was first pushed to fit-window after that PR had already been merged, so it comes as its own PR.)

  • Errors (App.svelte): the generated bindings type Go slices as Pad[] | null, because a nil Go slice encodes as null. Go always sends arrays, so App.svelte now reads st?.pads ?? [] once and uses that everywhere.
  • Warnings:
    • The Pads and Sequences panels were <main role="tabpanel">, which Svelte flags; they are now <div>s.
    • The About backdrop (a click outside the dialog closes it) had a svelte-ignore with its two codes separated by a space. Svelte 5 needs a comma, so the ignore didn't apply. Escape and the Close button already cover the keyboard.

No layout or behaviour changes.

Testing

The server has no Node, so svelte-check hasn't been run there. Please run on the laptop:

  1. npm run check in frontend: 0 errors and 0 warnings.
  2. wails3 dev: the app looks and behaves as before (both tabs, About opens and closes).