Thanks @fr3nzzy. Can you sign the CLA https://dev.brackets.io/brackets-contributor-license-agreement.html before we merge?

@jasonsanjose ofc, i will sign it tomorrow (i need to read it before, right?:)). If it's alright. I'm going to sleep now.

Turns out, it's very small CLA. Signed.

Done with code review.

Fixed. My bad.
But, also, i have a question about multiline comments. As far as i know, there are no multiline comments in VBScript. But it's been a while...

Good catch. Correct there are no block comments in VBScript. There is also the REM single-line comment statement.

I've removed block comments array for VBScript. Also REM added to list of line comments.

I'll merge this to get coloring for .vbs files and .asp files get HTML tag coloring.

Unfortunately, VBScript blocks in .asp file are not colored correctly, but it's not working for .aspx files, either.

Note that detecting script block delimiters is tricky because there are many delimiters types:

• <%@LANGUAGE="VBSCRIPT"%> at top of page with <% ... %> in page
• <script type="(text|application)\/(x-)?vb(a|script)">
• <script runat="server">

@redmunds Damn, i missed custom MIME declaration for "text/x-brackets-html". I can add <script type="(text|application)\/(x-)?vb(a|script)"> for "text/x-brackets-html" (it will enable VBScript highlighting in htmlmixed mode for <script type="text/vbscript"></script>.

Is there any way to enable <script runat="server"></script> and <%@LANGUAGE="VBSCRIPT"%>too? Because, right now it's partial solution and i want to do it right :) Can't reach codemirror.net for some reasons.

I already merged this pull request, so start a new one.

Since <%@LANGUAGE="VBSCRIPT"%> is specified at the top of the page, it won't work the same. You would have to assume that all <% ... %> and <script runat="server"> are VBScript (which is usually true).

To do it right I think you'd really want a new Language declaration specifically for asp/aspx -- probably based on CodeMirror's newer "htmlembedded" mode. We're shoehorning a lot of file types into the basic HTML mode right now not because it's the best answer, but just because no one has implemented the custom mode that's needed yet.