app/static/js/Forms/CreateContributionForm.js

@@ -1,8 +1,8 @@
-Forms.CreateContributionForm = class CreateContributionForm extends Forms.BaseForm {
+class CreateContributionForm extends Form {
   static autoInit() {
     let createContributionFormElements = document.querySelectorAll('.create-contribution-form');
     for (let createContributionFormElement of createContributionFormElements) {
-      new Forms.CreateContributionForm(createContributionFormElement);
+      new CreateContributionForm(createContributionFormElement);
     }
   }
 
@@ -15,4 +15,4 @@ Forms.CreateContributionForm = class CreateContributionForm extends Forms.BaseFo
       }
     });
   }
-};
+}

app/static/js/Forms/CreateCorpusFileForm.js

@@ -1,8 +1,8 @@
-Forms.CreateCorpusFileForm = class CreateCorpusFileForm extends Forms.BaseForm {
+class CreateCorpusFileForm extends Form {
   static autoInit() {
     let createCorpusFileFormElements = document.querySelectorAll('.create-corpus-file-form');
     for (let createCorpusFileFormElement of createCorpusFileFormElements) {
-      new Forms.CreateCorpusFileForm(createCorpusFileFormElement);
+      new CreateCorpusFileForm(createCorpusFileFormElement);
     }
   }
 
@@ -15,4 +15,4 @@ Forms.CreateCorpusFileForm = class CreateCorpusFileForm extends Forms.BaseForm {
       }
     });
   }
-};
+}

app/static/js/Forms/CreateJobForm.js

@@ -1,8 +1,8 @@
-Forms.CreateJobForm = class CreateJobForm extends Forms.BaseForm {
+class CreateJobForm extends Form {
   static autoInit() {
     let createJobFormElements = document.querySelectorAll('.create-job-form');
     for (let createJobFormElement of createJobFormElements) {
-      new Forms.CreateJobForm(createJobFormElement);
+      new CreateJobForm(createJobFormElement);
     }
   }
 
@@ -22,4 +22,4 @@ Forms.CreateJobForm = class CreateJobForm extends Forms.BaseForm {
       }
     });
   }
-};
+}

app/static/js/Forms/Form.js

@@ -1,17 +1,9 @@
-var Forms = {};
-
-Forms.autoInit = () => {
-  for (let propertyName in Forms) {
-    let property = Forms[propertyName];
-    // Call the autoInit method of all properties that are subclasses of Forms.BaseForm
-    if (property.prototype instanceof Forms.BaseForm) {
-      property.autoInit();
+class Form {
+  static autoInit() {
+    CreateContributionForm.autoInit();
+    CreateCorpusFileForm.autoInit();
+    CreateJobForm.autoInit();
   }
-  }
-};
-
-Forms.BaseForm = class BaseForm {
-  static autoInit() {throw 'Not implemented';}
 
   constructor(formElement) {
     this.formElement = formElement;
@@ -147,4 +139,4 @@ Forms.BaseForm = class BaseForm {
       }
     }
   }
-};
+}

app/static/js/ResourceDisplays/CorpusDisplay.js

@@ -1,4 +1,4 @@
-ResourceDisplays.CorpusDisplay = class CorpusDisplay extends ResourceDisplays.BaseDisplay {
+class CorpusDisplay extends ResourceDisplay {
   constructor(displayElement) {
     super(displayElement);
     this.corpusId = displayElement.dataset.corpusId;
@@ -102,4 +102,4 @@ ResourceDisplays.CorpusDisplay = class CorpusDisplay extends ResourceDisplays.Ba
       new Date(creationDate).toLocaleString("en-US")
     );
   }
-};
+}

app/static/js/ResourceDisplays/JobDisplay.js

@@ -1,4 +1,4 @@
-ResourceDisplays.JobDisplay = class JobDisplay extends ResourceDisplays.BaseDisplay {
+class JobDisplay extends ResourceDisplay {
   constructor(displayElement) {
     super(displayElement);
     this.jobId = this.displayElement.dataset.jobId;
@@ -123,4 +123,4 @@ ResourceDisplays.JobDisplay = class JobDisplay extends ResourceDisplays.BaseDisp
   setServiceVersion(serviceVersion) {
     this.setElements(this.displayElement.querySelectorAll('.job-service-version'), serviceVersion);
   }
-};
+}

app/static/js/ResourceDisplays/ResourceDisplay.js

@@ -1,6 +1,4 @@
-var ResourceDisplays = {};
-
-ResourceDisplays.BaseDisplay = class BaseDisplay {
+class ResourceDisplay {
   constructor(displayElement) {
     this.displayElement = displayElement;
     this.userId = this.displayElement.dataset.userId;
@@ -43,4 +41,4 @@ ResourceDisplays.BaseDisplay = class BaseDisplay {
       this.setElement(element, value);
     }
   }
-};
+}

app/static/js/ResourceLists/JobList.js

@@ -29,7 +29,7 @@ class JobList extends ResourceList {
 
   get item() {
     return `
-      <tr class="list-item service-scheme clickable hoverable">
+      <tr class="list-item service-scheme">
         <td>
           <label class="list-action-trigger" data-list-action="select">
             <input class="select-checkbox" type="checkbox">
@@ -112,7 +112,7 @@ class JobList extends ResourceList {
     if (listItemElement === null) {return;}
     let itemId = listItemElement.dataset.id;
     let listActionElement = event.target.closest('.list-action-trigger[data-list-action]');
-    let listAction = listActionElement === null ? 'view' : listActionElement.dataset.listAction;
+    let listAction = listActionElement === null ? '' : listActionElement.dataset.listAction;
     switch (listAction) {
       case 'delete-request': {
         let values = this.listjs.get('id', itemId)[0].values();

app/templates/_scripts.html.j2

@@ -35,20 +35,20 @@
 {%- assets
   filters='rjsmin',
   output='gen/Forms.%(version)s.js',
-  'js/forms/index.js',
-  'js/forms/create-contribution-form.js',
-  'js/forms/create-corpus-file-form.js',
-  'js/forms/create-job-form.js'
+  'js/Forms/Form.js',
+  'js/Forms/CreateCorpusFileForm.js',
+  'js/Forms/CreateJobForm.js',
+  'js/Forms/CreateContributionForm.js'
 %}
 <script src="{{ ASSET_URL }}"></script>
 {%- endassets %}
 
 {%- assets
   filters='rjsmin',
-  output='gen/resource-displays.%(version)s.js',
-  'js/resource-displays/index.js',
-  'js/resource-displays/corpus-display.js',
-  'js/resource-displays/job-display.js'
+  output='gen/ResourceDisplays.%(version)s.js',
+  'js/ResourceDisplays/ResourceDisplay.js',
+  'js/ResourceDisplays/CorpusDisplay.js',
+  'js/ResourceDisplays/JobDisplay.js'
 %}
 <script src="{{ ASSET_URL }}"></script>
 {%- endassets %}
@@ -141,7 +141,7 @@
     {alignment: 'right', constrainWidth: false, coverTrigger: false}
   );
   ResourceList.autoInit();
-  Forms.autoInit();
+  Form.autoInit();
 
   // Display flashed messages
   for (let [category, message] of {{ get_flashed_messages(with_categories=True)|tojson }}) {

app/templates/corpora/corpus.html.j2

@@ -237,7 +237,7 @@
 {% block scripts %}
 {{ super() }}
 <script>
-let corpusDisplay = new ResourceDisplays.CorpusDisplay(document.querySelector('#corpus-display'));
+let corpusDisplay = new CorpusDisplay(document.querySelector('#corpus-display'));
 
 {# {% if current_user.is_following_corpus(corpus) %}
   let unfollowRequestElement = document.querySelector('.action-button[data-action="unfollow-request"]');

app/templates/jobs/job.html.j2

@@ -150,7 +150,7 @@
 {% block scripts %}
 {{ super() }}
 <script>
-  let jobDisplay = new ResourceDisplays.JobDisplay(document.querySelector('#job-display'));
+  let jobDisplay = new JobDisplay(document.querySelector('#job-display'));
   let deleteJobRequestElement = document.querySelector('#delete-job-request');
   let restartJobRequestElement = document.querySelector('#restart-job-request');
   deleteJobRequestElement.addEventListener('click', (event) => {