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

Always direct the user to the line where the crime occurred #1113

Open
jidanni opened this issue Apr 25, 2024 · 0 comments
Open

Always direct the user to the line where the crime occurred #1113

jidanni opened this issue Apr 25, 2024 · 0 comments

Comments

@jidanni
Copy link
Contributor

jidanni commented Apr 25, 2024

Don't say

line 53 column 3 - Info: missing optional end tag </li>

Say instead

line 53 column 3 - Info: this <li> tag is missing its optional end tag </li> further down

Or say

line 53 column 3 - Info: this <li> tag is missing its optional end tag </li> further down, at line 1234 column 3 in fact

Or best say

line 1234 column 3 - Info: the <li> tag back up at line 53 column 3 is missing its optional end tag </li> here

because that's where the "crime" occurred. Anywhere else and the user will be scratching thier head when looking at the HTML.

Otherwise "Gas leak at the corner of 35th and Roberts." So we go there and see nothing and then open up the full report. Oh, he started smelling gas at 35th, but forgot to tell us the actual leak was at 42nd.

Here is a skeleton of a different incident. I'm not attaching sample files. I'm just making this skeleton.

    124:  </li>
    126:  <li>   <--user's attention is sent here,
    153:    <li>
    168:    </li>
    170:    <li>
    196:    </li>
    198:    <li>
    202:    </li>
    204:    <li>
    221:    </li>
    223:    <li>
    240:    </li>
    242:    <li>
    255:    </li>
    257:    <li>
    275:    </li>
    277:    <li>
    280:    </li>
    310:  <li>   <--- when all along the user forgot to add a </li> before this line, here!
    405:  </li>

(Acutually li>|[dou]l> would be a better pattern that I could have used to show the structure here.)

Same problem with <dd>

line 413 column 5 - Info: missing optional end tag </dd>

Sorry. still version 5.6 on Debian

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

No branches or pull requests

1 participant