search

ajaxorg / ace

Ace (Ajax.org Cloud9 Editor)
https://ace.c9.io
Other
26.69k stars 5.28k forks source link

XML Syntax Error Indication on a Wrong Line #3479

Open boghyon opened 6 years ago

boghyon commented 6 years ago

Tested Browsers / Devices

Windows 10

URL Minimal Example

https://ace.c9.io/build/kitchen-sink.html?doc=XML

Steps to Reproduce the Problem

  1. Open an XML document to edit
  2. Optional: If it's not already the case, separate a single-lined XML tag into multiple LOC
  3. Add a syntax error intentionally e.g. a character right after an attribute value: "..."s.

What Is the Expected Result?

ace editor xml expected error indication placement The error / warning indication is on the same line as where the error actually placed.

What Happens Instead?

ace editor xml wrong error indication placement

Any other information?

Even with an XML tag which has all the attributes in a single line, the indication is placed somewhere else which is weird. ace editor xml wrong error indication placement even with a single line I had a case where I made such a syntax error and didn't know where it was until I had to scroll down.

boghyon commented 6 years ago

I guess this issue is a duplicate of https://github.com/ajaxorg/ace/issues/2795. I'll close this and hope that https://github.com/ajaxorg/ace/issues/2795 gets fixed soon..

boghyon commented 2 years ago

@andrewnester I'm reopening this issue since the GitHub bot closed https://github.com/ajaxorg/ace/issues/2795. The issue is still reproducible.

github-actions[bot] commented 1 year ago

This issue has not received any attention in 1 year. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

boghyon commented 1 year ago

The issue is still reproducible. Do not close this issue.

nightwing commented 1 year ago

The github-actions bot configuration is a bit annoying indeed.

The built in linters most likely will be deprecated in favor of https://github.com/mkslanc/ace-linters, you can test that with https://ace.c9.io/kitchen-sink.html?doc=XML&useAceLinters=true it uses a different xml parser which doesn't have this problem.

github-actions[bot] commented 4 months ago

This issue has not received any attention in 1 year. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

boghyon commented 4 months ago

@nightwing It would be great if the linter functionality could be built-in without any additional third-party lib.