Diff action shows a "/" in first line about oxygen-xml-editor-xspec-support HOT 7 CLOSED

Also, the <test:ws> elements shouldn't be visible, right?

from oxygen-xml-editor-xspec-support.

Also, the test:ws elements shouldn't be visible, right?

I'm not sure what to do with them... They look like they contain the original spacing/indent but if that's the case I would have expected them to appear for each indent. If I look inside the intermediate *-result.xml file I see one such test:ws element between each two tags from the x:expect element, but in the x:result one there are text:ws elements missing (for example between the and element).

do you have any suggestions? Should I remove, unwrap or keep them as the only indent (and discard the other indent nodes).

from oxygen-xml-editor-xspec-support.

I'm not sure what to do with them... They look like they contain the original spacing/indent

That's what it is indeed: xspec/xspec#227 (comment)
But I also had to learn it in first place...

So what to do? I'm undecided... From a technical point I understand why it's important to have this information around, but from a UX perspective I was (and still am) a bit confused to see it...

Removing all other textnodes and unwrapping the original whitespace nodes seems like a nice idea...

@cirulls @AirQuick do you have an idea? Or should @AlexJitianu just leave the diff as is? I would be fine with it now that I know what the test:ws elements are for... but I could also image the solution mentioned above...

from oxygen-xml-editor-xspec-support.

The whitespace-only text node in test:ws is significant. So it shouldn't be removed.

We could just unwrap it. But the screenshot suggests that the problem happens on oXygen Diff, right? Then I recall that oXygen Diff has Format and Indent Both Files button (which itself is quite useful). If the user hits the button, I'm afraid the unwrapped whitespace-only text node is lost.

Another option might be unwrapping test:ws and replacing its whitespace-only text node with a string just like the HTML report does. In this case, the user may still wonder what this \n\r\t. string is... But I guess at least it isn't too ugly nor lost by Format and Indent.

from oxygen-xml-editor-xspec-support.

Hi,
For ignoring to test:ws you have to remove extra line and remove new line your input[wrap input element].

from oxygen-xml-editor-xspec-support.

Also, the <test:ws> elements shouldn't be visible, right?

Hi,
For ignoring to test:ws you have to remove extra line and remove new line your input[wrap input element].

from oxygen-xml-editor-xspec-support.

I've followed @AirQuick's suggestions and I've replaced test:ws with a string just like the HTML report does. The next plugin build will contain the fix.

from oxygen-xml-editor-xspec-support.