From 76119cb113f5489458b2a431eeb35fd0e4d06989 Mon Sep 17 00:00:00 2001
From: Filipa Lacerda <filipa@gitlab.com>
Date: Tue, 28 Mar 2017 18:01:47 +0100
Subject: Changes after review

---
 app/assets/javascripts/environments/components/environment_item.js  | 6 ++++--
 .../javascripts/environments/components/environments_table.js       | 2 +-
 .../javascripts/environments/services/environments_service.js       | 3 ++-
 3 files changed, 7 insertions(+), 4 deletions(-)

(limited to 'app/assets/javascripts/environments')

diff --git a/app/assets/javascripts/environments/components/environment_item.js b/app/assets/javascripts/environments/components/environment_item.js
index 3fe1042557e..9f297644368 100644
--- a/app/assets/javascripts/environments/components/environment_item.js
+++ b/app/assets/javascripts/environments/components/environment_item.js
@@ -425,16 +425,18 @@ export default {
   },
 
   template: `
-    <tr :class="{ 'child-row': model.isChildren }">
+    <tr>
       <td>
         <a v-if="!model.isFolder"
           class="environment-name"
+          :class="{ 'prepend-left-default': model.isChildren }"
           :href="environmentPath">
           {{model.name}}
         </a>
         <span v-else
           class="folder-name"
-          @click="onClickFolder">
+          @click="onClickFolder"
+          role="button">
 
           <span class="folder-icon">
             <i
diff --git a/app/assets/javascripts/environments/components/environments_table.js b/app/assets/javascripts/environments/components/environments_table.js
index 5a1f28d0bdf..0daa942edee 100644
--- a/app/assets/javascripts/environments/components/environments_table.js
+++ b/app/assets/javascripts/environments/components/environments_table.js
@@ -69,7 +69,7 @@ export default {
           <template v-if="model.isFolder && model.isOpen && model.children && model.children.length > 0">
             <tr v-if="isLoadingFolderContent">
               <td colspan="6" class="text-center">
-                <i class="fa fa-spin fa-spinner" aria-hidden="true"/>
+                <i class="fa fa-spin fa-spinner fa-2x" aria-hidden="true"/>
               </td>
             </tr>
 
diff --git a/app/assets/javascripts/environments/services/environments_service.js b/app/assets/javascripts/environments/services/environments_service.js
index 4917df64a56..ad8d9984561 100644
--- a/app/assets/javascripts/environments/services/environments_service.js
+++ b/app/assets/javascripts/environments/services/environments_service.js
@@ -18,6 +18,7 @@ export default class EnvironmentsService {
   }
 
   getFolderContent(folderUrl) {
-    return Vue.http.get(`${folderUrl}.json?per_page=3`);
+    const results = 3;
+    return Vue.http.get(`${folderUrl}.json?per_page=${results}`);
   }
 }
-- 
cgit v1.2.1