-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Completion for xsd:enumeration inside of text node #632
Completion for xsd:enumeration inside of text node #632
Conversation
To test this PR, see usecase at redhat-developer/vscode-xml#218 (comment)
Here a demo which shows in action xs:enumeration on text: |
Looks promising. Could you plug the "did you mean" feature in? I don't remember if we implemented it in lemminx |
521099f
to
3494976
Compare
This PR is ready (there are some tests), the only part which is not managed is the documentation for the xs:enumeration value (see the FIXME). I don't know how to manage that for the moment. I think we should create an another issue to manage that and support hover together. |
What do you mean with
No it was not implemented and my PR implements that. |
895d691
to
c312f0c
Compare
Yes please don't open it:) The problem with this XML Schema is that it generates too many symbols (you can exclude the symbols for this files in the settings), but I'm glad that you have seen the problem and now I think you will understand why redhat-developer/vscode-xml#237 is very important :) |
That's strange, travis doesnt fail? |
b3dd576
to
0d2c486
Compare
Fixes redhat-developer/vscode-xml#218 Signed-off-by: azerr <[email protected]>
@xorye please retry |
It works! |
I see. I still get this problem when I disable symbols/exclude the file. I can investigate more while taking care of redhat-developer/vscode-xml#237 |
imported XSD Fixes eclipse-lemminx#632 Signed-off-by: azerr <[email protected]>
xsd:enumeration autocomplete don't work for text node
Fixes redhat-developer/vscode-xml#218
Signed-off-by: azerr [email protected]