Skip to content
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

Fix error range for cvc-complex-type-2.3 #908

Merged

Conversation

datho7561
Copy link
Contributor

@datho7561 datho7561 commented Oct 9, 2020

Now handles text nodes that are only take up one line, as well as <![CDATA[]]>.

Closes #885

Signed-off-by: David Thompson [email protected]

@datho7561
Copy link
Contributor Author

Sorry, I don't think I quite understand the second part. Is this so that it highlights between the pipes in this case?

<elementsOnly>
  |text content
  <child />
  more text content|
</elementsOnly>

@datho7561 datho7561 force-pushed the 885-cvc-complex-type-2-3 branch from 3c0d174 to e7a530c Compare October 15, 2020 14:26
@datho7561
Copy link
Contributor Author

^fixed the first case

@angelozerr
Copy link
Contributor

Sorry, I don't think I quite understand the second part. Is this so that it highlights between the pipes in this case?

I think you will not have cvc-complex-type-2.3 error in this case.

@datho7561 datho7561 force-pushed the 885-cvc-complex-type-2-3 branch from e7a530c to 3b2dd72 Compare October 15, 2020 15:03
Now handles text nodes that are only take up one line,
as well as <![CDATA[]]>.

Closes eclipse-lemminx#885

Signed-off-by: David Thompson <[email protected]>
@datho7561 datho7561 force-pushed the 885-cvc-complex-type-2-3 branch from 3b2dd72 to af94419 Compare October 15, 2020 15:04
@datho7561
Copy link
Contributor Author

Thanks for the help, Angelo! I avoided using getData(), and now the ranges don't include whitespace in CDATA sections.

@angelozerr angelozerr merged commit af09659 into eclipse-lemminx:master Oct 15, 2020
@angelozerr
Copy link
Contributor

Works like a charm. Great job @datho7561 !

@datho7561 datho7561 deleted the 885-cvc-complex-type-2-3 branch December 22, 2020 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error range for cvc-complex-type.2.3
2 participants