Backport PR #6653: Fix IPython.utils.ansispan() to ignore stray [0m

Previously, `IPython.utils.fixConsole("\033[0m")` (I use `fixConsole()` because `ansispan()` is private) gave `"</span>"`. I also changed `ansiyellow` to something that looks actually yellow (but still dark). I'm submitting this to `master`, but it would be good to backport this to 2.x (whatever the procedure is).

IPython/html/tests/base/utils.js

@@ -0,0 +1,23 b''
+casper.notebook_test(function () {
+    var input = [
+        "\033[0m[\033[0minfo\033[0m] \033[0mtext\033[0m",
+        "\033[0m[\033[33mwarn\033[0m] \033[0m\tmore text\033[0m",
+        "\033[0m[\033[33mwarn\033[0m] \033[0m  https://some/url/to/a/file.ext\033[0m",
+        "\033[0m[\033[31merror\033[0m] \033[0m\033[0m",
+        "\033[0m[\033[31merror\033[0m] \033[0m\teven more text\033[0m",
+        "\033[0m[\033[31merror\033[0m] \033[0m\t\tand more more text\033[0m"].join("\n");
+
+    var output = [
+        "[info] text",
+        "[<span  class=\"ansiyellow\">warn</span>] \tmore text",
+        "[<span  class=\"ansiyellow\">warn</span>]   https://some/url/to/a/file.ext",
+        "[<span  class=\"ansired\">error</span>] ",
+        "[<span  class=\"ansired\">error</span>] \teven more text",
+        "[<span  class=\"ansired\">error</span>] \t\tand more more text"].join("\n");
+
+    var result = this.evaluate(function (input) {
+        return IPython.utils.fixConsole(input);
+    }, input);
+
+    this.test.assertEquals(result, output, "IPython.utils.fixConsole() handles [0m correctly");
+});

IPython/html/static/base/js/utils.js

@@ -290,11 +290,19 b' IPython.utils = (function (IPython) {'
     function ansispan(str) {
         // ansispan function adapted from github.com/mmalecki/ansispan (MIT License)
         // regular ansi escapes (using the table above)
+        var is_open = false
         return str.replace(/\033\[(0?[01]|22|39)?([;\d]+)?m/g, function(match, prefix, pattern) {
             if (!pattern) {
                 // [(01|22|39|)m close spans
+                if (is_open) {
+                    is_open = false;
                 return "</span>";
+                } else {
+                    return "";
             }
+            } else {
+                is_open = true;
+
             // consume sequence of color escapes
             var numbers = pattern.match(/\d+/g);
             var attrs = {};
@@ -308,6 +316,7 b' IPython.utils = (function (IPython) {'
                 span = span + " " + attr + '="' + attrs[attr] + '"';
             }
             return span + ">";
+            }
         });
     };
     

IPython/html/static/notebook/less/ansicolors.less

@@ -1,13 +1,12 b''
 /* CSS font colors for translated ANSI colors. */
 
-
 .ansibold {font-weight: bold;}
 
 /* use dark versions for foreground, to improve visibility */
 .ansiblack {color: black;}
 .ansired {color: darkred;}
 .ansigreen {color: darkgreen;}
-.ansiyellow {color: brown;}
+.ansiyellow {color: #c4a000;}
 .ansiblue {color: darkblue;}
 .ansipurple {color: darkviolet;}
 .ansicyan {color: steelblue;}
@@ -22,4 +21,3 b''
 .ansibgpurple {background-color: magenta;}
 .ansibgcyan {background-color: cyan;}
 .ansibggray {background-color: gray;}
-

IPython/html/static/style/ipython.min.css

@@ -31,7 +31,7 b' div.traceback-wrapper{text-align:left;max-width:800px;margin:auto}'
 .ansiblack{color:#000}
 .ansired{color:#8b0000}
 .ansigreen{color:#006400}
-.ansiyellow{color:#a52a2a}
+.ansiyellow{color:#c4a000}
 .ansiblue{color:#00008b}
 .ansipurple{color:#9400d3}
 .ansicyan{color:#4682b4}

IPython/html/static/style/style.min.css

@@ -1336,7 +1336,7 b' input.engine_num_input{width:60px}'
 .ansiblack{color:#000}
 .ansired{color:#8b0000}
 .ansigreen{color:#006400}
-.ansiyellow{color:#a52a2a}
+.ansiyellow{color:#c4a000}
 .ansiblue{color:#00008b}
 .ansipurple{color:#9400d3}
 .ansicyan{color:#4682b4}