fix: emit sl-change from checkbox on user interaction only

[bennypowers]

fixes #497

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/shoelace/shoelace/AV1K4r5QQYnaVRaNzpcJ7QZVS8Fk
✅ Preview: https://shoelace-git-fork-bennypowers-fix-checkbox-emit-414397-shoelace.vercel.app

[claviska]

Nice catch, and thanks so much for the tests!

While we're looking at this, it probably makes sense to add the same logic to <sl-switch> and possibly <sl-radio>. Is that something you're interested in adding to this? (I'm happy to tackle it in the next day or two if you don't have the time.)

[bennypowers]

sure i can grab that

[claviska]

Thanks again. I noticed <sl-switch> wasn't emitting sl-change using left/right arrows, so I fixed that and added a test. Unfortunately, I don't think I'm using sendKeys() right here because I couldn't get it to recognize the arrow keys.

3bb8614#diff-dc93b9977ec07e71d499241926223e79068a694b82e6c1b81664aefc28149917R38-R47

I don't have time to dig deeper at the moment so I commented it out. Maybe I'm using the wrong string, but I assumed it uses the same values as event.key.

[bennypowers]

yeah that's def. the correct string. odd
https://playwright.dev/docs/api/class-keyboard#keyboard-down