Fix app.View constructors

Author: simon04

assets/javascripts/views/content/content.js

@@ -9,22 +9,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.Content = class Content extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.scrollToTop = this.scrollToTop.bind(this);
-    this.scrollToBottom = this.scrollToBottom.bind(this);
-    this.scrollStepUp = this.scrollStepUp.bind(this);
-    this.scrollStepDown = this.scrollStepDown.bind(this);
-    this.scrollPageUp = this.scrollPageUp.bind(this);
-    this.scrollPageDown = this.scrollPageDown.bind(this);
-    this.onReady = this.onReady.bind(this);
-    this.onBootError = this.onBootError.bind(this);
-    this.onEntryLoading = this.onEntryLoading.bind(this);
-    this.onEntryLoaded = this.onEntryLoaded.bind(this);
-    this.beforeRoute = this.beforeRoute.bind(this);
-    this.afterRoute = this.afterRoute.bind(this);
-    this.onClick = this.onClick.bind(this);
-    this.onAltF = this.onAltF.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -50,6 +36,21 @@ app.views.Content = class Content extends app.View {
   }
 
   init() {
+    this.scrollToTop = this.scrollToTop.bind(this);
+    this.scrollToBottom = this.scrollToBottom.bind(this);
+    this.scrollStepUp = this.scrollStepUp.bind(this);
+    this.scrollStepDown = this.scrollStepDown.bind(this);
+    this.scrollPageUp = this.scrollPageUp.bind(this);
+    this.scrollPageDown = this.scrollPageDown.bind(this);
+    this.onReady = this.onReady.bind(this);
+    this.onBootError = this.onBootError.bind(this);
+    this.onEntryLoading = this.onEntryLoading.bind(this);
+    this.onEntryLoaded = this.onEntryLoaded.bind(this);
+    this.beforeRoute = this.beforeRoute.bind(this);
+    this.afterRoute = this.afterRoute.bind(this);
+    this.onClick = this.onClick.bind(this);
+    this.onAltF = this.onAltF.bind(this);
+
     this.scrollEl = app.isMobile()
       ? document.scrollingElement || document.body
       : this.el;

assets/javascripts/views/content/entry_page.js

@@ -10,14 +10,8 @@
 (function () {
   let LINKS = undefined;
   app.views.EntryPage = class EntryPage extends app.View {
-    constructor(...args) {
-      super(...args);
-      this.beforeRoute = this.beforeRoute.bind(this);
-      this.onSuccess = this.onSuccess.bind(this);
-      this.onError = this.onError.bind(this);
-      this.onClick = this.onClick.bind(this);
-      this.onAltC = this.onAltC.bind(this);
-      this.onAltO = this.onAltO.bind(this);
+    constructor() {
+      super();
     }
 
     static initClass() {
@@ -40,6 +34,13 @@
     }
 
     init() {
+      this.beforeRoute = this.beforeRoute.bind(this);
+      this.onSuccess = this.onSuccess.bind(this);
+      this.onError = this.onError.bind(this);
+      this.onClick = this.onClick.bind(this);
+      this.onAltC = this.onAltC.bind(this);
+      this.onAltO = this.onAltO.bind(this);
+
       this.cacheMap = {};
       this.cacheStack = [];
     }

assets/javascripts/views/content/offline_page.js

@@ -6,9 +6,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.OfflinePage = class OfflinePage extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.onClick = this.onClick.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -20,6 +19,10 @@ app.views.OfflinePage = class OfflinePage extends app.View {
     };
   }
 
+  init() {
+    this.onClick = this.onClick.bind(this);
+  }
+
   deactivate() {
     if (super.deactivate(...arguments)) {
       this.empty();

assets/javascripts/views/content/root_page.js

@@ -5,8 +5,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.RootPage = class RootPage extends app.View {
-  constructor(...args) {
-    super(...args);
+  constructor() {
+    super();
     this.onClick = this.onClick.bind(this);
   }
 

assets/javascripts/views/content/settings_page.js

@@ -7,10 +7,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.SettingsPage = class SettingsPage extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.onChange = this.onChange.bind(this);
-    this.onClick = this.onClick.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -22,6 +20,11 @@ app.views.SettingsPage = class SettingsPage extends app.View {
     };
   }
 
+  init() {
+    this.onChange = this.onChange.bind(this);
+    this.onClick = this.onClick.bind(this);
+  }
+
   render() {
     this.html(this.tmpl("settingsPage", this.currentSettings()));
   }

assets/javascripts/views/layout/document.js

@@ -5,10 +5,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.Document = class Document extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.afterRoute = this.afterRoute.bind(this);
-    this.onVisibilityChange = this.onVisibilityChange.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -28,6 +26,9 @@ app.views.Document = class Document extends app.View {
   }
 
   init() {
+    this.afterRoute = this.afterRoute.bind(this);
+    this.onVisibilityChange = this.onVisibilityChange.bind(this);
+
     this.addSubview(
       (this.menu = new app.views.Menu()),
       this.addSubview((this.sidebar = new app.views.Sidebar()))

assets/javascripts/views/layout/menu.js

@@ -4,9 +4,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.Menu = class Menu extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.onGlobalClick = this.onGlobalClick.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -17,6 +16,8 @@ app.views.Menu = class Menu extends app.View {
   }
 
   init() {
+    this.onGlobalClick = this.onGlobalClick.bind(this);
+
     $.on(document.body, "click", this.onGlobalClick);
   }
 

assets/javascripts/views/layout/mobile.js

@@ -52,6 +52,9 @@ app.views.Mobile = class Mobile extends app.View {
 
   constructor() {
     super(document.documentElement);
+  }
+
+  init() {
     this.showSidebar = this.showSidebar.bind(this);
     this.hideSidebar = this.hideSidebar.bind(this);
     this.onClickBack = this.onClickBack.bind(this);
@@ -62,10 +65,7 @@ app.views.Mobile = class Mobile extends app.View {
     this.onTapSearch = this.onTapSearch.bind(this);
     this.onEscape = this.onEscape.bind(this);
     this.afterRoute = this.afterRoute.bind(this);
-    this.el = document.documentElement;
-  }
 
-  init() {
     $.on($("._search"), "touchend", this.onTapSearch);
 
     this.toggleSidebar = $("button[data-toggle-sidebar]");

assets/javascripts/views/layout/path.js

@@ -5,10 +5,8 @@
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
 app.views.Path = class Path extends app.View {
-  constructor(...args) {
-    super(...args);
-    this.onClick = this.onClick.bind(this);
-    this.afterRoute = this.afterRoute.bind(this);
+  constructor() {
+    super();
   }
 
   static initClass() {
@@ -20,6 +18,11 @@ app.views.Path = class Path extends app.View {
     this.routes = { after: "afterRoute" };
   }
 
+  init() {
+    this.onClick = this.onClick.bind(this);
+    this.afterRoute = this.afterRoute.bind(this);
+  }
+
   render(...args) {
     this.html(this.tmpl("path", ...Array.from(args)));
     this.show();

assets/javascripts/views/layout/resizer.js

@@ -8,11 +8,8 @@
   let MIN = undefined;
   let MAX = undefined;
   app.views.Resizer = class Resizer extends app.View {
-    constructor(...args) {
-      super(...args);
-      this.onDragStart = this.onDragStart.bind(this);
-      this.onDrag = this.onDrag.bind(this);
-      this.onDragEnd = this.onDragEnd.bind(this);
+    constructor() {
+      super();
     }
 
     static initClass() {
@@ -32,6 +29,10 @@
     }
 
     init() {
+      this.onDragStart = this.onDragStart.bind(this);
+      this.onDrag = this.onDrag.bind(this);
+      this.onDragEnd = this.onDragEnd.bind(this);
+
       this.el.setAttribute("draggable", "true");
       this.appendTo($("._app"));
     }