From ae0f9637994531aee7c0d972f883755f0676b64b Mon Sep 17 00:00:00 2001
From: Daniel Imms <daimms@microsoft.com>
Date: Thu, 2 Jun 2016 02:17:03 -0700
Subject: [PATCH] Remove extra whitespace from class names

Use [].join to retain browser compatibility

Fixes #73
---
 src/xterm.js | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/xterm.js b/src/xterm.js
index 608f88b..73ddfa1 100644
--- a/src/xterm.js
+++ b/src/xterm.js
@@ -1040,7 +1040,7 @@
               if (data === -1) {
                 out += '<span class="reverse-video terminal-cursor">';
               } else {
-                out += '<span class="';
+                var classNames = [];
 
                 bg = data & 0x1ff;
                 fg = (data >> 9) & 0x1ff;
@@ -1048,18 +1048,18 @@
 
                 if (flags & Terminal.flags.BOLD) {
                   if (!Terminal.brokenBold) {
-                    out += ' xterm-bold ';
+                    classNames.push('xterm-bold');
                   }
                   // See: XTerm*boldColors
                   if (fg < 8) fg += 8;
                 }
 
                 if (flags & Terminal.flags.UNDERLINE) {
-                  out += ' xterm-underline ';
+                  classNames.push('xterm-underline');
                 }
 
                 if (flags & Terminal.flags.BLINK) {
-                    out += ' xterm-blink ';
+                  classNames.push('xterm-blink');
                 }
 
                 /**
@@ -1074,7 +1074,7 @@
                 }
 
                 if (flags & Terminal.flags.INVISIBLE) {
-                  out += ' xterm-hidden ';
+                  classNames.push('xterm-hidden');
                 }
 
                 /**
@@ -1094,14 +1094,18 @@
                 }
 
                 if (bg < 256) {
-                  out += ' xterm-bg-color-' + bg + ' ';
+                  classNames.push('xterm-bg-color-' + bg);
                 }
 
                 if (fg < 256) {
-                  out += ' xterm-color-' + fg + ' ';
+                  classNames.push('xterm-color-' + fg);
                 }
 
-                out += '">';
+                out += '<span';
+                if (classNames.length) {
+                  out += ' class="' + classNames.join(' ') + '"';
+                }
+                out += '>';
               }
             }
           }