Remove automatic init call on page load from ardublockly.js.

Importing a js file should not have any secondary effects, like automatically running code. This also allows other Ardublockly-based applications to reuse a lot of the Ardublockly front end code and have their own onLoad init.

ardublockly/ardublockly.js

@@ -21,9 +21,8 @@ Ardublockly.LANGUAGE_NAME = {
  */
 Ardublockly.LANG = 'en';
 
-/** Initialize function for Ardublockly on page load. */
-window.addEventListener('load', function load(event) {
-  window.removeEventListener('load', load, false);
+/** Initialize function for Ardublockly, to be called on page load. */
+Ardublockly.init = function() {
   // Lang init must run first for the rest of the page to pick the right msgs
   Ardublockly.initLanguage();
 
@@ -43,7 +42,7 @@ window.addEventListener('load', function load(event) {
       document.location.hostname != '192.168.0.7') {
     Ardublockly.openNotConnectedModal();
   }
-});
+};
 
 /** Binds functions to each of the buttons, nav links, and related. */
 Ardublockly.bindActionFunctions = function() {

ardublockly/index.html

@@ -228,7 +228,7 @@
       <div class="settings_div">
         <label>Default IDE button:</label> <br>
         <select name="settings_ide" id="ide_settings">
-          <option value="verify">IDE ptions unknown</option> 
+          <option value="verify">IDE ptions unknown</option>
         </select>
       </div>
       <div class="settings_div">
@@ -298,5 +298,11 @@
   <script src="ardublockly_design.js"></script>
   <script src="ardublockly_blockly.js"></script>
   <script src="ardublockly.js"></script>
+  <script>
+    window.addEventListener('load', function load(event) {
+      window.removeEventListener('load', load, false);
+      Ardublockly.init();
+    });
+  </script>
 </body>
 </html>