Move code for the debug trace window to an external script

Chrome doesn't allow inline scripts, only external ones
7 years ago · 08a54197c0
parent b98d7a8f3b
commit 08a54197c0
4 changed files with 16 additions and 10 deletions
--- a/gulpfile.js
+++ b/gulpfile.js
@ -125,6 +125,10 @@ sources.receiverJs = [
    './tabs/receiver_msp.js'
 ];

+sources.debugTraceJs = [
+    './js/debug_trace.js'
+];
+
 sources.hexParserJs = [
    './js/workers/hex_parser.js',
 ];
@ -136,6 +140,7 @@ var output = {
    mapJs: 'map.js',
    receiverCss: 'receiver-msp.css',
    receiverJs: 'receiver-msp.js',
+    debugTraceJs: 'debug-trace.js',
    hexParserJs: 'hex_parser.js',
 };

--- a/js/debug_trace.js
+++ b/js/debug_trace.js
@ -0,0 +1,9 @@
+function debugTraceOnLoad()
+{
+    var output = document.getElementById('debug-trace');
+    setInterval(function() {
+        output.innerText = getDebugTrace();
+    }, 100);
+}
+
+window.onload = debugTraceOnLoad;
--- a/src/css/tabs/sensors.css
+++ b/src/css/tabs/sensors.css
@ -172,9 +172,8 @@
    float: right;
    margin-right: 1em;
    font-size: 90%;
-    text-decoration: underline;
 }

-.tab-sensors a.debug-trace::hover {
+.tab-sensors a.debug-trace:hover {
    text-decoration: underline;
 }
--- a/tabs/debug_trace.html
+++ b/tabs/debug_trace.html
@ -1,14 +1,7 @@
 <html>
 <head>
 <title>Debug Trace</title>
-<script type="text/javascript">
-    window.onload = function() {
-        var output = document.getElementById('debug-trace');
-        setInterval(function() {
-            output.innerText = getDebugTrace();
-        }, 100);
-    }
-</script>
+    <script type="text/javascript" src="/build/debug-trace.js"></script>
 </head>
 <body>
    <pre><code id="debug-trace"></code></pre>