typst-preview.nvim icon indicating copy to clipboard operation
typst-preview.nvim copied to clipboard

[DISCUSION] some feature/issue disscussion

Open AbaoFromCUG opened this issue 1 year ago • 1 comments

Background

  • Enter-tainer/typst-preview have been archived and consolidated into tinymist
  • Some flags be deprecated/removed, e.g. --static-file-host is deprecated

I already read...

  • https://github.com/chomosuke/typst-preview.nvim/pull/34
  • README.md
  • https://enter-tainer.github.io/typst-preview/standalone.html
  • https://myriad-dreamin.github.io/tinymist/feature/preview.html#loc-1x20.00x864.63
  • https://github.com/Enter-tainer/typst-preview/pull/148

I would like to

  • Support static host:port spec, include Address already in use erorr display
  • Add optional neoconf per project configuration, to config host:port (address)
  • Re-annotation config type with LuaCATS (lua_ls style) comment annotation
  • Clarify spawn preveiw args, e.g. remove depreacated flags
  • Remind user when compile fails and websocket disconnects (like vscode status bar)

I am willing to contribution PR, here want to discuss specific requirements

AbaoFromCUG avatar Nov 09 '24 15:11 AbaoFromCUG

Hi, thank you for wanting to contribute to this plugin.

I'm happy with all changes you're proposing. Note that:

  • Address already in use will need to respond correctly to the situation where the port is already used by another unrelated to neovim application. This must be done by inspection the output of tinymist.

Happy to hear about any of your plans before you go ahead and implement them.

chomosuke avatar Nov 19 '24 07:11 chomosuke