perf(UI): improved connection status indicator

package.json

@@ -105,7 +105,7 @@
     "node-sass": "^5.0.0",
     "sass-loader": "^10.1.1",
     "standard-version": "^9.2.0",
-    "stylelint": "^13.9.0",
+    "stylelint": "^13.12.0",
     "stylelint-config-standard": "^21.0.0",
     "stylelint-scss": "^3.19.0",
     "vue": "^2.6.12",

src/renderer/components/TheSettingBar.vue

@@ -19,7 +19,7 @@
                   @contextmenu.prevent="contextMenu($event, connection)"
                   @mouseover.self="tooltipPosition"
                >
-                  <i class="settingbar-element-icon dbi" :class="`dbi-${connection.client} ${connected.includes(connection.uid) ? 'badge' : ''}`" />
+                  <i class="settingbar-element-icon dbi" :class="`dbi-${connection.client} ${getStatusBadge(connection.uid)}`" />
                   <span class="ex-tooltip-content">{{ getConnectionName(connection.uid) }}</span>
                </li>
             </draggable>
@@ -73,7 +73,7 @@ export default {
       ...mapGetters({
          getConnections: 'connections/getConnections',
          getConnectionName: 'connections/getConnectionName',
-         connected: 'workspaces/getConnected',
+         getWorkspace: 'workspaces/getWorkspace',
          selectedWorkspace: 'workspaces/getSelected',
          updateStatus: 'application/getUpdateStatus'
       }),
@@ -109,6 +109,20 @@ export default {
          const el = e.target;
          const fromTop = window.pageYOffset + el.getBoundingClientRect().top - (el.offsetHeight / 4);
          el.querySelector('.ex-tooltip-content').style.top = `${fromTop}px`;
+      },
+      getStatusBadge (uid) {
+         const status = this.getWorkspace(uid).connection_status;
+
+         switch (status) {
+            case 'connected':
+               return 'badge badge-connected';
+            case 'connecting':
+               return 'badge badge-connecting';
+            case 'failed':
+               return 'badge badge-failed';
+            default:
+               return '';
+         }
       }
    }
 };

src/renderer/components/Workspace.vue

@@ -1,7 +1,7 @@
 <template>
    <div v-show="isSelected" class="workspace column columns col-gapless">
       <WorkspaceExploreBar :connection="connection" :is-selected="isSelected" />
-      <div v-if="workspace.connected" class="workspace-tabs column columns col-gapless">
+      <div v-if="workspace.connection_status === 'connected'" class="workspace-tabs column columns col-gapless">
          <ul
             id="tabWrap"
             ref="tabWrap"

src/renderer/components/WorkspaceExploreBar.vue

@@ -8,7 +8,7 @@
       >
          <div class="workspace-explorebar-header">
             <span class="workspace-explorebar-title">{{ connectionName }}</span>
-            <span v-if="workspace.connected" class="workspace-explorebar-tools">
+            <span v-if="workspace.connection_status === 'connected'" class="workspace-explorebar-tools">
                <i
                   class="mdi mdi-18px mdi-database-plus c-hand mr-2"
                   :title="$t('message.createNewSchema')"
@@ -28,7 +28,7 @@
             </span>
          </div>
          <div class="workspace-explorebar-search">
-            <div v-if="workspace.connected" class="has-icon-right">
+            <div v-if="workspace.connection_status === 'connected'" class="has-icon-right">
                <input
                   v-model="searchTerm"
                   class="form-input input-sm"
@@ -39,7 +39,7 @@
             </div>
          </div>
          <WorkspaceConnectPanel
-            v-if="!workspace.connected"
+            v-if="workspace.connection_status !== 'connected'"
             class="workspace-explorebar-body"
             :connection="connection"
          />

src/renderer/scss/_transitions.scss

@@ -39,3 +39,17 @@
     transform: scale(1);
   }
 }
+
+@keyframes connecting {
+  0% {
+    background-color: transparent;
+  }
+
+  50% {
+    background-color: $warning-color;
+  }
+
+  100% {
+    background-color: transparent;
+  }
+}

src/renderer/scss/main.scss

@@ -136,6 +136,20 @@ body {
       box-shadow: none;
     }
   }
+
+  &.badge-connected::after {
+    background: $success-color;
+  }
+
+  &.badge-connecting::after {
+    animation-name: connecting;
+    animation-duration: 2s;
+    animation-iteration-count: infinite;
+  }
+
+  &.badge-failed::after {
+    background: $error-color;
+  }
 }
 
 .form-select {

src/renderer/scss/themes/dark-theme.scss

@@ -386,7 +386,6 @@
             bottom: -10px;
             right: 0;
             position: absolute;
-            background: $success-color;
           }
 
           &.badge-update::after {

src/renderer/scss/themes/light-theme.scss

@@ -168,7 +168,6 @@
             bottom: -10px;
             right: 0;
             position: absolute;
-            background: $success-color;
           }
 
           &.badge-update::after {

src/renderer/store/modules/workspaces.store.js

@@ -37,7 +37,7 @@ export default {
       },
       getConnected: state => {
          return state.workspaces
-            .filter(workspace => workspace.connected)
+            .filter(workspace => workspace.connection_status === 'connected')
             .map(workspace => workspace.uid);
       },
       getLoadedSchemas: state => uid => {
@@ -54,7 +54,7 @@ export default {
       SELECT_WORKSPACE (state, uid) {
          state.selected_workspace = uid;
       },
-      ADD_CONNECTED (state, payload) {
+      SET_CONNECTED (state, payload) {
          const { uid, client, dataTypes, indexTypes, customizations, structure, version } = payload;
 
          state.workspaces = state.workspaces.map(workspace => workspace.uid === uid
@@ -65,19 +65,41 @@ export default {
                indexTypes,
                customizations,
                structure,
-               connected: true,
+               connection_status: 'connected',
                version
             }
             : workspace);
       },
-      REMOVE_CONNECTED (state, uid) {
+      SET_CONNECTING (state, uid) {
          state.workspaces = state.workspaces.map(workspace => workspace.uid === uid
             ? {
                ...workspace,
                structure: {},
                breadcrumbs: {},
                loaded_schemas: new Set(),
-               connected: false
+               connection_status: 'connecting'
+            }
+            : workspace);
+      },
+      SET_FAILED (state, uid) {
+         state.workspaces = state.workspaces.map(workspace => workspace.uid === uid
+            ? {
+               ...workspace,
+               structure: {},
+               breadcrumbs: {},
+               loaded_schemas: new Set(),
+               connection_status: 'failed'
+            }
+            : workspace);
+      },
+      SET_DISCONNECTED (state, uid) {
+         state.workspaces = state.workspaces.map(workspace => workspace.uid === uid
+            ? {
+               ...workspace,
+               structure: {},
+               breadcrumbs: {},
+               loaded_schemas: new Set(),
+               connection_status: 'disconnected'
             }
             : workspace);
       },
@@ -247,10 +269,14 @@ export default {
          commit('SELECT_WORKSPACE', uid);
       },
       async connectWorkspace ({ dispatch, commit }, connection) {
+         commit('SET_CONNECTING', connection.uid);
+
          try {
             const { status, response } = await Connection.connect(connection);
-            if (status === 'error')
+            if (status === 'error') {
                dispatch('notifications/addNotification', { status, message: response }, { root: true });
+               commit('SET_FAILED', connection.uid);
+            }
             else {
                let dataTypes = [];
                let indexTypes = [];
@@ -288,7 +314,7 @@ export default {
                   dispatch('connections/editConnection', connProxy, { root: true });
                }
 
-               commit('ADD_CONNECTED', {
+               commit('SET_CONNECTED', {
                   uid: connection.uid,
                   client: connection.client,
                   dataTypes,
@@ -382,13 +408,13 @@ export default {
       },
       removeConnected ({ commit }, uid) {
          Connection.disconnect(uid);
-         commit('REMOVE_CONNECTED', uid);
+         commit('SET_DISCONNECTED', uid);
          commit('SELECT_TAB', { uid, tab: 0 });
       },
       addWorkspace ({ commit, dispatch, getters }, uid) {
          const workspace = {
             uid,
-            connected: false,
+            connection_status: 'disconnected',
             selected_tab: 0,
             search_term: '',
             tabs: [],