Fix error range for cvc-datatype-valid-1-2-1

[NikolasKomonen]

If only integers are allowed as content and you have:

<a xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" 
   xsi:noNamespaceSchemaLocation="intType.xsd">

  TEXT
  

</a>

with XSD:

<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema">
  <xs:element name="a" type="xs:integer">
  </xs:element>
</xs:schema>

The highlight could be cleaned up to the first and last character of text here:

And in the case a user puts in a tag the highlight, error message should be adjusted to include the first tag it comes across:

[angelozerr]

@NikolasKomonen could you please add a simple xml schema and simple xml in this issue to reproduce quickly your issue. It will help too when tests will be written. Thanks!

[angelozerr]

@xorye do you think you could work on this issue for 0.8.0? Otherwise we will change the millestone.

[xorye]

@angelozerr working on this now

[xorye]

@angelozerr @NikolasKomonen If there is only whitespace and newlines, is it okay if the error range looks like this?

[xorye]

Any opinions on this case? This case is when there is whitespace in the middle of text.

[fbricon]

compare with eclipse/idea

[xorye]

Eclipse looks like this:

Which is what we have already, except that eclipse ignores the newline character.

IntelliJ looks like this:

IntelliJ only underlines the parent element.

The IntelliJ way is definitely easier to implement but it's not what was initially desired. (Unless that's open to change)

Referring back to the two screenshots I posted yesterday, should I just keep them as is, for now?

[fbricon]

yeah keep as-is