Error lines is only applied to the column instead of the whole line about flycheck HOT 12 CLOSED

This is intentional.

from flycheck.

Mmm ok so I guess you are aiming for granularity. Is there a way I can get the whole token highlighted instead of just the column?

from flycheck.

I'll make it customizable.

from flycheck.

👍

from flycheck.

Just discovered another problem with this change, sometimes checkers like jshint will report an error on a column that is a space, e.g a defined but unused variable, jshint will report the column that is the space right after the unused variable. This change doesn't highlight spaces in the middle of a line. This may affect PR #39 too, as some checkers report trailing whitespace errors, but that's not highlighted in PR #39.

from flycheck.

Sorry, but I fail to understand what you're trying to tell me. I guess the jshint checker somehow highlights the wrong column or so. If so that is a bug. In this case please report it as new issue, and include the following in your description:

• The source code that is checked.
• The output of the checker if run from command line with the same arguments flycheck uses.
• A screenshot of the highlighting you observe
• A description of the highlighting that you'd expect, at best an exact range of positions that you'd like to see highlighted.

from flycheck.

Error Message:

lib/backbone-pageable.js: line 399, col 23, 'lastPage' is defined but never used.

The point is where the error column is. Currently it is not hightlighted because it's a space. It's not an error from jshint, I'd expect that space to be hightlighted by flycheck.

from flycheck.

I'm sorry but that is not something Flycheck can fix. The default faces of Emacs are simply ill-suited to highlight errors, because they lack “catchiness” and cannot highlight spaces though spaces can actually be errors (think of trailing spaces that are forbidden by many style guides and hence warned about by many syntax checkers).

Really, really use reasonable faces that have a striking background color and/or an underline, like in the screenshot in README.md.

from flycheck.

I'll document that, though.

from flycheck.

Ah right. Good deal.

from flycheck.

Do you think dd0fe96 provides enough information?

from flycheck.

Note that I changed the customization setting to disable column-specific highlighting.

Replace (setq flycheck-ignore-columns t) with (setq flycheck-highlighting-mode 'lines).

from flycheck.