Toggle Line Comment in LESS files uses block comments instead

[peterflynn]

1. Select a few lines of code in a LESS file
2. Hit Ctrl+/

Result: the code is commented using /* ... */

Expected: commented using // -- this is how Brackets works in Sprint 37

@njx Could this be due to your EditorCommandHandler changes? Toggle Line Comment still works in JS code, so maybe not -- maybe something changed in cmv4's LESS mode that makes LanguageManager confused about whether it's CSS vs. LESS under the cursor?

[redmunds]

I think this is because we do not yet have true less mode support -- we just use css mode for .less files. Nevermind -- it does work correctly in Sprint 37.

[peterflynn]

Yeah, we've used a LESS-specific mode for LESS files forever. The particular mode we use did change semi-recently (in Sprint 36), but as pointed out above this worked fine in Sprint 36 & 37 -- it's only broken on current master.

[njx]

Turns out this was working semi-accidentally before, due to the fact that the way getModeForSelection() calculates modes happened to just return the mime type we set on the whole document in this case, whereas now we're relying on a more granular function that tries to figure out the mode of the selection range, and it's just returning the CSS mode that's configured for LESS. For now, we'll do a hack similar to what we do for HTML to assume that "xml"-mode content inside an HTML document is actually HTML; longer-term we should try to get CM to be able to report back the mime type for a given mode configuration.

[peterflynn]

Confirmed fixed