"mixed space and tabs in line ..." errors/warnings should be hideable completely

[knbknb]

"mixed space and tabs in line ..." errors/warnings should be hideable completely

[dascritch]

I wrote about this to Mr Crockford. He responded me this:
«Those are all bad practices. JSLint is trying to help you, if you will let it.»

So no, i don't think this is configurable yet ;)

[GarrettS]

why would you ever want to allow tabs and spaces together? I've always hated that so much.

[knbknb]

In my personal development workflow, I tolerate mixed spaces and tabs, because:

When I am done, or about to commit to SVN/git, I send the javascript code through a pretty-printer, (v8 for example has a commandline option for this), which turns tabs into spaces. However, I much more often run my JS code through JSlint, which produces lots of these mixed-space-and-tabs warnings which I find so distracting. This output turns attention away from other issues which are more important (superfluous trailing commas for instance).

In any case, this was just a whishlist feature, or enhancement request, that came to mind after I read about this project on hacker news.

[GarrettS]

I understand you asked for the option but to me, mixing TAB and space sounds like a really bad option. I still don't see why y9ou mix TAB with space?

Mixing is a problem because many editors' TAB width is four spaces sometimes it is 8 spaces or or otherwise. Autoformatters may convert TAB to a certain number of spaces, too, e.g. 2, 4, 8, or even 3.

When TAB and space are mixed, what looks good in one editor usually will look improperly indented in another.

[knbknb]

I don't mix tab and space deliberately, but it just happens, e.g when I indent a selection blockwise (Shift+Tab in eclipse) .

It is reported as an error. Error messages should be reserved for something causing really serious trouble. Mixed spaces and tabs aren't errors; I consider this a warning at best.

[GarrettS]

Eclipse, yah, there is definitely a way to change TAB so that it indends with spaces but it depends on which JS editor you use. I'll show you how the built-in one works. Launching eclipse, that'll take a couple mins...

window > preferences
. > General > Editors > File Associations... ".js" --> JavaScript Editor (default)

Next:
. > JavaScript > Code Style > Formatter > Edit... "Tab Policy: Spaces only"

Next:
. > JavaScript > Editor > Typing, checkbox: [*] Tab key adjusts the indentation of the current line.

[derekdata]

This should be a warning instead of an error, as to the best of my knowledge it does not impact the execution of the code.

JSHint should focus on things that break the code, not on codestyle issues. Otherwise we might as well just use JSLint

[timmywil]

Maybe this common comment banner could be an exception to the tabs/spaces rule:

function foo() {
    /**
TAB  * @private
TAB  */
    function bar() {
        // something
    }
}

I've just hit exactly the same thing as timmywil notes. I'm getting hundreds of errors just because I like the 's in my comments to line up. I'd say an exception to the rule should be something like "\t? *".

Just hit an extension to this issue as well - my comments are being flagged up for having tailing white space when I've got something like:

*{space}

Might it be possible to not check tailing white space in comments?

[timmywil]

Reason for closing?