From ef9f228f933257ed9f3b7cdf420bdcb055778bb0 Mon Sep 17 00:00:00 2001
From: Michael Lange <dingoeatingfuzz@gmail.com>
Date: Mon, 11 Feb 2019 14:37:36 -0800
Subject: [PATCH] Changes from code review

---
 ui/app/controllers/jobs/index.js | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/ui/app/controllers/jobs/index.js b/ui/app/controllers/jobs/index.js
index 659c8e5d0..bd27f7d9a 100644
--- a/ui/app/controllers/jobs/index.js
+++ b/ui/app/controllers/jobs/index.js
@@ -51,6 +51,16 @@ export default Controller.extend(Sortable, Searchable, {
   fuzzySearchProps: computed(() => ['name']),
   fuzzySearchEnabled: true,
 
+  qpType: '',
+  qpStatus: '',
+  qpDatacenter: '',
+  qpPrefix: '',
+
+  selectionType: qpSelection('qpType'),
+  selectionStatus: qpSelection('qpStatus'),
+  selectionDatacenter: qpSelection('qpDatacenter'),
+  selectionPrefix: qpSelection('qpPrefix'),
+
   optionsType: computed(() => [
     { key: 'batch', label: 'Batch' },
     { key: 'parameterized', label: 'Parameterized' },
@@ -125,18 +135,8 @@ export default Controller.extend(Sortable, Searchable, {
     }));
   }),
 
-  qpType: '',
-  qpStatus: '',
-  qpDatacenter: '',
-  qpPrefix: '',
-
-  selectionType: qpSelection('qpType'),
-  selectionStatus: qpSelection('qpStatus'),
-  selectionDatacenter: qpSelection('qpDatacenter'),
-  selectionPrefix: qpSelection('qpPrefix'),
-
   /**
-    Filtered jobs are those that match the selected namespace and aren't children
+    Visible jobs are those that match the selected namespace and aren't children
     of periodic or parameterized jobs.
   */
   visibleJobs: computed('model.[]', 'model.@each.parent', function() {