lsp: Offer to populate empty documents #4767
Conversation
|
Of course this only shows while the file does not contain anything but whitespace:-) |
There was a problem hiding this comment.
Nice. I like it.
I think only one option is enough though.
Probably the Hello World like so:
import { AboutSlint, VerticalBox } from "std-widgets.slint";
export component MainWindow inherits Window {
VerticalBox {
alignment: start;
Text { text: "Hello World!"; }
AboutSlint { preferred-height: 150px; }
}
}
Make it a window by default. But keep it as short as possible while still have something somehow visual.
Did you try if it worked in a slint!{} macro?
| if let Some(current_version) = version { | ||
| if current_version != source_version { | ||
| return Err( | ||
| "Document version mismatch. Please refresh your command data".into() |
There was a problem hiding this comment.
This is the kind of error that is seen by the user when something goes wrong in synchronization. But users will have no idea how to "refresh their command data".
There was a problem hiding this comment.
Well, when that happens the LSP stops anyway :-)
I'll just return "OK" and be done with that...
| if let Some(range) = util::map_node(node) { | ||
| let has_non_ws_token = node | ||
| .children_with_tokens() | ||
| .any(|nt| nt.kind() != SyntaxKind::Whitespace && nt.kind() != SyntaxKind::Eof); |
There was a problem hiding this comment.
Should we also accept a comment? (eg license header comments)
There was a problem hiding this comment.
Did you try if it worked in a slint!{} macro?
Yes and that fails hard:-/
It replaces the entire rust code.
|
We need to remember which area of the document we are actually touching... otherwise this just can not work. |
So when snippets do not work: Let's try something else to get started from scratch:-)