Merge pull request #350 from microsoft/xterm4

Update example xterm to v4
microsoft · Sep 11, 2019 · d15af01 · d15af01
2 parents 347f5dd + 0815402
commit d15af01
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/examples/electron/index.html b/examples/electron/index.html
@@ -3,7 +3,7 @@
   <head>
     <meta charset="UTF-8">
     <title>node-pty Electron example</title>
-    <link rel="Stylesheet" href="./node_modules/xterm/lib/xterm.css">
+    <link rel="Stylesheet" href="./node_modules/xterm/css/xterm.css">
   </head>
   <body>
     <div id="xterm"></div>

diff --git a/examples/electron/package.json b/examples/electron/package.json
@@ -11,6 +11,6 @@
   "dependencies": {
     "electron": "4.0.1",
     "node-pty": "^0.8.0",
-    "xterm": "^3.14.0"
+    "xterm": "^4.0.0"
   }
 }
diff --git a/examples/electron/renderer.js b/examples/electron/renderer.js
@@ -17,9 +17,7 @@ const xterm = new Terminal();
 xterm.open(document.getElementById('xterm'));
 
 // Setup communication between xterm.js and node-pty
-xterm.on('data', (data) => {
-  ptyProcess.write(data);
-});
+xterm.onData(data => ptyProcess.write(data));
 ptyProcess.on('data', function (data) {
   xterm.write(data);
 });