Highlight ranges positioned incorrectly when using tabs about linter HOT 10 CLOSED

Nice catch @postcasio !

from linter.

@park9140 Do you have time to look into this issue? Or I can do it :)

from linter.

@postcasio, @iam4x, this is a bug in the jshint linter implementation and or in jshint itself. Unlike other editors that report tabs as single character in width jshint consistantly seems to report that a tab is in fact 4 characters. This shifts the reported positions. After a bit of investigation I can't seem to find a way to prevent jshint from doing this.

It is going to be decently complicated to adjust the positioning properly in the jshint linter. I have reported the issue to jshint, jshint/jshint#1714

from linter.

Spoke to soon. Found a work around. In your .jshintrc set indent: 1. This will cause jshint to assume that your tab width is 1 and then report the columns correctly.

from linter.

Reading further it looks like version 2.5 change log calls this out explicitly as the new functionality.

https://github.com/jshint/jshint/releases/tag/2.5.0

So... we should probably just write this off as the known fix when you are using tabs and not spaces.

That release basically makes jshint useless to me in daily usage. We should get a javascript style linter built to restore the style linting they just gutted out of jshint.

from linter.

👍 nice findings!

from linter.

Thanks for your help @park9140, adding indent: 1 works great.

from linter.

@park9140 Nice finds! I use jscs as code style linter for JavaScript.

from linter.

Seems like there really isn't anything we can do about this with the current state of jshint. I'm going to close this.

Workaround is to just add 'indent: 1' to your jshint rules.

If anyone determines this doesn't work or they add a method to set this configuration from command line re-open and we can work from there.

from linter.

Can anyone add this note about indent: 1 to the documentation? I had to dig through the issues to find this info.

from linter.