Enter-tainer / typst-preview

[DEPRECATED] Use tinymist instead
https://Enter-tainer.github.io/typst-preview/
MIT License
444 stars 21 forks source link

feat: auto scroll sync by keyboard events #233

Closed Myriad-Dreamin closed 10 months ago

Myriad-Dreamin commented 10 months ago

The description onSelectionChange is bad, since it is actually "when selection changes by mouse", but we may keep compability. Regardless of compability, the perfect enums are probably:

"onSelectionChange" -> "onSelectionChangeByMouse"
"onAnyChange" -> "onSelectionChange"
Enter-tainer commented 10 months ago

compability concern shouldnt be a big problem. i guess most users still use the default one. so if we also change the default config while renaming enums, users shouldnt notice the change.

IMO renaming this to the perfect state is okay.