From a3925c33712257fea2c6e6513e488cd350118cf9 Mon Sep 17 00:00:00 2001
From: Anthony Lapenna <anthony.lapenna@portainer.io>
Date: Tue, 11 Aug 2020 12:39:01 +1200
Subject: [PATCH] feat(k8s/sidebar): relocate setup entry as a sub-entry
 (#4192)

---
 .../kubernetesSidebarContent.html                           | 6 +++---
 .../kubernetes-sidebar-content/kubernetesSidebarContent.js  | 1 +
 app/portainer/views/sidebar/sidebar.html                    | 3 +++
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.html b/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.html
index 226a1f5d7..b83f8c2f9 100644
--- a/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.html
+++ b/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.html
@@ -15,7 +15,7 @@
 </li>
 <li class="sidebar-list">
   <a ui-sref="kubernetes.cluster({endpointId: $ctrl.endpointId})" ui-sref-active="active">Cluster <span class="menu-icon fa fa-server fa-fw"></span></a>
-</li>
-<li class="sidebar-list" ng-if="$ctrl.adminAccess">
-  <a ui-sref="portainer.endpoints.endpoint.kubernetesConfig({id: $ctrl.endpointId})">Setup <span class="menu-icon fa fa-cog fa-fw"></span></a>
+  <div class="sidebar-sublist" ng-if="$ctrl.adminAccess && ($ctrl.currentState === 'kubernetes.cluster' || $ctrl.currentState === 'portainer.endpoints.endpoint.kubernetesConfig')">
+    <a ui-sref="portainer.endpoints.endpoint.kubernetesConfig({id: $ctrl.endpointId})">Setup</a>
+  </div>
 </li>
diff --git a/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.js b/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.js
index 8b74d4a47..8a44e8a8d 100644
--- a/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.js
+++ b/app/kubernetes/components/kubernetes-sidebar-content/kubernetesSidebarContent.js
@@ -3,5 +3,6 @@ angular.module('portainer.kubernetes').component('kubernetesSidebarContent', {
   bindings: {
     adminAccess: '<',
     endpointId: '<',
+    currentState: '<',
   },
 });
diff --git a/app/portainer/views/sidebar/sidebar.html b/app/portainer/views/sidebar/sidebar.html
index efdba0a39..012e86ab4 100644
--- a/app/portainer/views/sidebar/sidebar.html
+++ b/app/portainer/views/sidebar/sidebar.html
@@ -17,6 +17,7 @@
         ng-if="applicationState.endpoint.mode && applicationState.endpoint.mode.provider === 'KUBERNETES'"
         admin-access="isAdmin"
         endpoint-id="endpointId"
+        current-state="$state.current.name"
       >
       </kubernetes-sidebar-content>
       <azure-sidebar-content ng-if="applicationState.endpoint.mode && applicationState.endpoint.mode.provider === 'AZURE'" endpoint-id="endpointId"> </azure-sidebar-content>
@@ -156,6 +157,7 @@
               $state.current.name === 'portainer.endpoints.endpoint' ||
               $state.current.name === 'portainer.endpoints.new' ||
               $state.current.name === 'portainer.endpoints.endpoint.access' ||
+              $state.current.name === 'portainer.endpoints.endpoint.kubernetesConfig' ||
               $state.current.name === 'portainer.groups' ||
               $state.current.name === 'portainer.groups.group' ||
               $state.current.name === 'portainer.groups.group.access' ||
@@ -173,6 +175,7 @@
               $state.current.name === 'portainer.endpoints.endpoint' ||
               $state.current.name === 'portainer.endpoints.new' ||
               $state.current.name === 'portainer.endpoints.endpoint.access' ||
+              $state.current.name === 'portainer.endpoints.endpoint.kubernetesConfig' ||
               $state.current.name === 'portainer.groups' ||
               $state.current.name === 'portainer.groups.group' ||
               $state.current.name === 'portainer.groups.group.access' ||