WIP: slint-lsp open to open the preview standalone
#7576
Conversation
| args: super::Cli, | ||
| path: std::path::PathBuf, | ||
| component: Option<String>, | ||
| ) -> crate::common::Result<()> { |
There was a problem hiding this comment.
Why do we need a fake language server?
It feels easier to extend the live-preview to request the files it did not find than to do an entire extra parsing run to find those... And that is the only functionality we really need from the LS at this point in time.
|
Seems rather too complicated to me. The only thing the live preview needs the LS for is the discovery of files and it could pretty easily report files it did not find, so that this is not needed anymore. Parsing the entire project just for that one extra time seems a bit much. It also needs the "editor" to update the text... which should not take too long, so you can probably do that synchronously just fine. |
Because we need to first parse all the files in the LSP thread to fill up the document cache to be able to send all the files to the preview. But this is just a quick and dirty test. I didn't want to spend too much time on it. I don't intend to merge this.
100% agree |
(Currently just a prototype WIP)
The idea is that you can do
slint-lsp open foo.slintand that opens the preview window, from which you can also edit the .slint file by changing property or drag and drop, but this is not connected to an editor.
It could also refresh by a file system watcher (not implemented yet)
The usecase is: