From f4036be52b30f86f51f373a8299b590cedac26ae Mon Sep 17 00:00:00 2001
From: Fabio Manganiello <fabio@manganiello.tech>
Date: Sat, 22 Apr 2023 16:31:47 +0200
Subject: [PATCH] Extracted and refactored more common elements of the Entity
 components.

---
 .../src/components/panels/Entities/Cpu.vue    |  4 --
 .../components/panels/Entities/CpuInfo.vue    | 51 --------------
 .../components/panels/Entities/CpuTimes.vue   |  6 --
 .../src/components/panels/Entities/Device.vue |  6 --
 .../src/components/panels/Entities/Disk.vue   | 53 +--------------
 .../src/components/panels/Entities/Entity.vue | 28 ++++----
 .../panels/Entities/MemoryStats.vue           | 55 +---------------
 .../panels/Entities/NetworkInterface.vue      | 66 ++-----------------
 .../components/panels/Entities/common.scss    | 58 ++++++++++++++++
 9 files changed, 76 insertions(+), 251 deletions(-)

diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/Cpu.vue b/platypush/backend/http/webapp/src/components/panels/Entities/Cpu.vue
index 1651837e..e5e72ff8 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/Cpu.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/Cpu.vue
@@ -38,9 +38,5 @@ export default {
       font-weight: bold;
     }
   }
-
-  .icon {
-    margin-right: 1em;
-  }
 }
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/CpuInfo.vue b/platypush/backend/http/webapp/src/components/panels/Entities/CpuInfo.vue
index 5d68e4ec..c3b1fa3b 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/CpuInfo.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/CpuInfo.vue
@@ -166,55 +166,4 @@ export default {
 
 <style lang="scss" scoped>
 @import "common";
-
-.entity {
-  .head {
-    padding: 0.25em;
-
-    .icon {
-      margin-right: 1em;
-    }
-  }
-}
-
-.collapse-toggler {
-  display: flex;
-  align-items: center;
-  flex: 1;
-  min-height: 3em;
-  cursor: pointer;
-
-  &:hover {
-    color: $default-hover-fg;
-  }
-}
-
-.attributes .child {
-  margin: 0 -0.5em;
-  padding: 0.5em 1em;
-
-  &:not(:last-child) {
-    border-bottom: 1px solid $border-color-1;
-  }
-
-  &:hover {
-    cursor: initial;
-  }
-
-  .label {
-    font-weight: bold;
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-
-  .value {
-    font-size: 0.95em;
-    text-align: right;
-
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-}
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/CpuTimes.vue b/platypush/backend/http/webapp/src/components/panels/Entities/CpuTimes.vue
index 8c845e68..2ddcdc5f 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/CpuTimes.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/CpuTimes.vue
@@ -28,10 +28,4 @@ export default {
 
 <style lang="scss" scoped>
 @import "common";
-
-.entity {
-  .icon {
-    margin-right: 1em;
-  }
-}
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/Device.vue b/platypush/backend/http/webapp/src/components/panels/Entities/Device.vue
index b599e53d..0d824cdb 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/Device.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/Device.vue
@@ -28,10 +28,4 @@ export default {
 
 <style lang="scss" scoped>
 @import "common";
-
-.device-container {
-  .icon {
-    margin-right: 1em;
-  }
-}
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/Disk.vue b/platypush/backend/http/webapp/src/components/panels/Entities/Disk.vue
index 58c0eca0..3614c48b 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/Disk.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/Disk.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="entity disk-container">
+  <div class="entity disk-container" :class="{expanded: !isCollapsed}">
     <div class="head" @click.stop="isCollapsed = !isCollapsed">
       <div class="col-1 icon">
         <EntityIcon
@@ -169,57 +169,6 @@ export default {
       text-align: right;
       font-weight: bold;
     }
-
-    .icon {
-      margin-right: 1em;
-    }
-  }
-}
-
-.collapse-toggler {
-  display: flex;
-  align-items: center;
-  flex: 1;
-  min-height: 3em;
-  cursor: pointer;
-
-  @include from($tablet) {
-    @include until($desktop) {
-      margin-left: 3.25em;
-    }
-  }
-
-  &:hover {
-    color: $default-hover-fg;
-  }
-}
-
-.attributes .child {
-  margin: 0 -0.5em;
-  padding: 0.5em 1em;
-
-  &:not(:last-child) {
-    border-bottom: 1px solid $border-color-1;
-  }
-
-  &:hover {
-    cursor: initial;
-  }
-
-  .label {
-    font-weight: bold;
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-
-  .value {
-    font-size: 0.95em;
-    text-align: right;
-
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
   }
 }
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/Entity.vue b/platypush/backend/http/webapp/src/components/panels/Entities/Entity.vue
index 893086fc..954c1c2f 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/Entity.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/Entity.vue
@@ -186,6 +186,12 @@ export default {
   }
 }
 
+@mixin expanded-entity {
+  background: $selected-bg;
+  font-weight: bold;
+  box-shadow: 0 0 3px 2px $default-shadow-color;
+}
+
 .entity-container {
   width: 100%;
   display: flex;
@@ -195,35 +201,23 @@ export default {
   border-bottom: $default-border-3;
 
   &.with-children:not(.collapsed) {
-    background: $selected-bg;
-    font-weight: bold;
-    box-shadow: 0 0 3px 2px $default-shadow-color;
+    @include expanded-entity();
   }
 
   &:hover {
     background: $hover-bg;
   }
 
-  .collapse-toggler {
-    display: flex;
-    justify-content: center;
-    align-items: center;
-    flex: 1;
-    min-height: 3em;
-    margin-left: 0;
-    cursor: pointer;
-
-    &:hover {
-      color: $default-hover-fg;
-    }
-  }
-
   .adjuster {
     cursor: pointer;
   }
 }
 
 :deep(.entity-container) {
+  .entity.expanded {
+    @include expanded-entity();
+  }
+
   .head {
     .name {
       display: inline-flex;
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/MemoryStats.vue b/platypush/backend/http/webapp/src/components/panels/Entities/MemoryStats.vue
index 411fa945..6b2b28a5 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/MemoryStats.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/MemoryStats.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="entity memory-stats-container">
+  <div class="entity memory-stats-container" :class="{expanded: !isCollapsed}">
     <div class="head" @click.stop="isCollapsed = !isCollapsed">
       <div class="col-1 icon">
         <EntityIcon
@@ -127,63 +127,10 @@ export default {
 
 .entity {
   .head {
-    padding: 0.25em;
-
-    .icon {
-      margin-right: 1em;
-    }
-
     .value {
       text-align: right;
       font-weight: bold;
     }
   }
 }
-
-.collapse-toggler {
-  display: flex;
-  align-items: center;
-  flex: 1;
-  min-height: 3em;
-  cursor: pointer;
-
-  @include from($tablet) {
-    @include until($desktop) {
-      margin-left: 3.25em;
-    }
-  }
-
-  &:hover {
-    color: $default-hover-fg;
-  }
-}
-
-.attributes .child {
-  margin: 0 -0.5em;
-  padding: 0.5em 1em;
-
-  &:not(:last-child) {
-    border-bottom: 1px solid $border-color-1;
-  }
-
-  &:hover {
-    cursor: initial;
-  }
-
-  .label {
-    font-weight: bold;
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-
-  .value {
-    font-size: 0.95em;
-    text-align: right;
-
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-}
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/NetworkInterface.vue b/platypush/backend/http/webapp/src/components/panels/Entities/NetworkInterface.vue
index 8401558d..4fa3ae6a 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/NetworkInterface.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/NetworkInterface.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="entity network-interface-container">
+  <div class="entity network-interface-container" :class="{expanded: !isCollapsed}">
     <div class="head" @click.stop="isCollapsed = !isCollapsed">
       <div class="col-1 icon">
         <EntityIcon
@@ -12,7 +12,7 @@
         <div class="name" v-text="value.name" />
       </div>
 
-      <div class="col-1 collapse-toggler" @click.stop="isCollapsed = !isCollapsed">
+      <div class="col-1 collapse-toggler">
         <i class="fas"
           :class="{'fa-chevron-down': isCollapsed, 'fa-chevron-up': !isCollapsed}" />
       </div>
@@ -91,7 +91,8 @@
         </div>
       </div>
 
-    <div class="child head" @click.stop="areAddressesCollapsed = !areAddressesCollapsed">
+    <div class="child head" :class="{expanded: !areAddressesCollapsed}"
+      @click.stop="areAddressesCollapsed = !areAddressesCollapsed">
       <div class="col-11 label">Addresses</div>
       <div class="col-1 collapse-toggler pull-right">
         <i class="fas"
@@ -105,7 +106,7 @@
           v-for="address in (value.addresses || [])"
           :key="address.address"
         >
-          <div class="child head"
+          <div class="child head" :class="{expanded: displayedAddresses[address.address]}"
             @click.stop="displayedAddresses[address.address] = !displayedAddresses[address.address]"
           >
             <div class="col-11 label" v-text="address.address" />
@@ -160,61 +161,4 @@ export default {
 
 <style lang="scss" scoped>
 @import "common";
-
-.entity {
-  .head {
-    padding: 0.25em;
-
-    .icon {
-      margin-right: 1em;
-    }
-  }
-}
-
-.collapse-toggler {
-  display: flex;
-  align-items: center;
-  flex: 1;
-  min-height: 3em;
-  cursor: pointer;
-
-  @include from($tablet) {
-    @include until($desktop) {
-      margin-left: 3.25em;
-    }
-  }
-
-  &:hover {
-    color: $default-hover-fg;
-  }
-}
-
-.attributes .child {
-  margin: 0 -0.5em;
-  padding: 0.5em 1em;
-
-  &:not(:last-child) {
-    border-bottom: 1px solid $border-color-1;
-  }
-
-  &:hover {
-    cursor: initial;
-  }
-
-  .label {
-    font-weight: bold;
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-
-  .value {
-    font-size: 0.95em;
-    text-align: right;
-
-    @include from($tablet) {
-      @extend .col-m-6;
-    }
-  }
-}
 </style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Entities/common.scss b/platypush/backend/http/webapp/src/components/panels/Entities/common.scss
index f89de230..1938e186 100644
--- a/platypush/backend/http/webapp/src/components/panels/Entities/common.scss
+++ b/platypush/backend/http/webapp/src/components/panels/Entities/common.scss
@@ -10,6 +10,12 @@
     display: flex;
     align-items: center;
     padding: 0.75em 0.25em;
+    min-height: 3.5em;
+    position: relative;
+
+    .icon {
+      margin-right: 1em;
+    }
 
     .label {
       margin-top: 0.25em;
@@ -38,6 +44,7 @@
     background: linear-gradient(0deg, $default-bg-5, $default-bg-2);
     border-top: 1px solid $border-color-1;
     box-shadow: $border-shadow-bottom;
+    font-weight: normal;
   }
 
   button {
@@ -51,3 +58,54 @@
     }
   }
 }
+
+.collapse-toggler {
+  position: absolute;
+  right: 0;
+  display: flex;
+  align-items: center;
+  justify-content: flex-end;
+  flex: 1;
+  min-height: 2em;
+  margin-right: 1.25em;
+  cursor: pointer;
+
+  &:hover {
+    color: $default-hover-fg;
+  }
+}
+
+.attributes .child {
+  padding: 0.5em 1em;
+
+  &:not(:last-child) {
+    border-bottom: 1px solid $border-color-1;
+  }
+
+  &:hover {
+    cursor: initial;
+  }
+
+  &.head {
+    cursor: pointer;
+    &:hover {
+      background: $hover-bg;
+    }
+  }
+
+  .label {
+    font-weight: bold;
+    @include from($tablet) {
+      @extend .col-m-6;
+    }
+  }
+
+  .value {
+    font-size: 0.95em;
+    text-align: right;
+
+    @include from($tablet) {
+      @extend .col-m-6;
+    }
+  }
+}