From 4f70a70dda5295a733c7344ea559beeece855768 Mon Sep 17 00:00:00 2001
From: LouisLam <louislam@users.noreply.github.com>
Date: Thu, 19 Aug 2021 18:05:14 +0800
Subject: [PATCH] splite the left monitor list into a component

---
 src/components/MonitorList.vue | 133 ++++++++++++++++++++++++++++++++
 src/pages/Dashboard.vue        | 135 ++-------------------------------
 src/pages/DashboardHome.vue    |   2 +-
 3 files changed, 140 insertions(+), 130 deletions(-)
 create mode 100644 src/components/MonitorList.vue

diff --git a/src/components/MonitorList.vue b/src/components/MonitorList.vue
new file mode 100644
index 000000000..496ad7a12
--- /dev/null
+++ b/src/components/MonitorList.vue
@@ -0,0 +1,133 @@
+<template>
+    <div class="shadow-box list mb-4">
+        <div v-if="Object.keys($root.monitorList).length === 0" class="text-center mt-3">
+            No Monitors, please <router-link to="/add">add one</router-link>.
+        </div>
+
+        <router-link v-for="(item, index) in sortedMonitorList" :key="index" :to="monitorURL(item.id)" class="item" :class="{ 'disabled': ! item.active }" @click="$root.cancelActiveList">
+            <div class="row">
+                <div class="col-6 col-md-8 small-padding" :class="{ 'monitorItem': $root.userHeartbeatBar == 'bottom' || $root.userHeartbeatBar == 'none' }">
+                    <div class="info">
+                        <Uptime :monitor="item" type="24" :pill="true" />
+                        {{ item.name }}
+                    </div>
+                </div>
+                <div v-show="$root.userHeartbeatBar == 'normal'" :key="$root.userHeartbeatBar" class="col-6 col-md-4">
+                    <HeartbeatBar size="small" :monitor-id="item.id" />
+                </div>
+            </div>
+
+            <div v-if="$root.userHeartbeatBar == 'bottom'" class="row">
+                <div class="col-12">
+                    <HeartbeatBar size="small" :monitor-id="item.id" />
+                </div>
+            </div>
+        </router-link>
+    </div>
+</template>
+
+<script>
+import HeartbeatBar from "../components/HeartbeatBar.vue";
+import Uptime from "../components/Uptime.vue";
+export default {
+    components: {
+        Uptime,
+        HeartbeatBar,
+    },
+    computed: {
+        sortedMonitorList() {
+            let result = Object.values(this.$root.monitorList);
+
+            result.sort((m1, m2) => {
+
+                if (m1.active !== m2.active) {
+                    if (m1.active === 0) {
+                        return 1;
+                    }
+
+                    if (m2.active === 0) {
+                        return -1;
+                    }
+                }
+
+                if (m1.weight !== m2.weight) {
+                    if (m1.weight > m2.weight) {
+                        return -1;
+                    }
+
+                    if (m1.weight < m2.weight) {
+                        return 1;
+                    }
+                }
+
+                return m1.name.localeCompare(m2.name);
+            })
+
+            return result;
+        },
+    },
+    methods: {
+        monitorURL(id) {
+            return "/dashboard/" + id;
+        },
+    },
+}
+</script>
+
+<style lang="scss" scoped>
+@import "../assets/vars.scss";
+
+.small-padding {
+    padding-left: 5px !important;
+    padding-right: 5px !important;
+}
+
+.list {
+    height: auto;
+    min-height: calc(100vh - 240px);
+
+    .item {
+        display: block;
+        text-decoration: none;
+        padding: 13px 15px 10px 15px;
+        border-radius: 10px;
+        transition: all ease-in-out 0.15s;
+
+        &.disabled {
+            opacity: 0.3;
+        }
+
+        .info {
+            white-space: nowrap;
+            overflow: hidden;
+            text-overflow: ellipsis;
+        }
+
+        &:hover {
+            background-color: $highlight-white;
+        }
+
+        &.active {
+            background-color: #cdf8f4;
+        }
+    }
+}
+
+.dark {
+    .list {
+        .item {
+            &:hover {
+                background-color: $dark-bg2;
+            }
+
+            &.active {
+                background-color: $dark-bg2;
+            }
+        }
+    }
+}
+
+.monitorItem {
+    width: 100%;
+}
+</style>
diff --git a/src/pages/Dashboard.vue b/src/pages/Dashboard.vue
index 5db704085..707017931 100644
--- a/src/pages/Dashboard.vue
+++ b/src/pages/Dashboard.vue
@@ -1,38 +1,13 @@
 <template>
     <div class="container-fluid">
         <div class="row">
-            <div class="col-12 col-md-5 col-xl-4">
-                <div v-if="! $root.isMobile">
+            <div v-if="! $root.isMobile" class="col-12 col-md-5 col-xl-4">
+                <div>
                     <router-link to="/add" class="btn btn-primary mb-3"><font-awesome-icon icon="plus" /> Add New Monitor</router-link>
                 </div>
-
-                <div v-if="showList" class="shadow-box list mb-4">
-                    <div v-if="Object.keys($root.monitorList).length === 0" class="text-center mt-3">
-                        No Monitors, please <router-link to="/add">add one</router-link>.
-                    </div>
-
-                    <router-link v-for="(item, index) in sortedMonitorList" :key="index" :to="monitorURL(item.id)" class="item" :class="{ 'disabled': ! item.active }" @click="$root.cancelActiveList">
-                        <div class="row">
-                            <div class="col-6 col-md-8 small-padding" v-bind:class="{ 'monitorItem': $root.userHeartbeatBar == 'bottom' || $root.userHeartbeatBar == 'none' }">
-                                <div class="info">
-                                    <Uptime :monitor="item" type="24" :pill="true" />
-                                    {{ item.name }}
-                                </div>
-                            </div>
-                            <div class="col-6 col-md-4" v-show="$root.userHeartbeatBar == 'normal'" :key="$root.userHeartbeatBar">
-                                <HeartbeatBar size="small" :monitor-id="item.id" />
-                            </div>
-                        </div>
-
-                        <div class="row" v-if="$root.userHeartbeatBar == 'bottom'">
-                            <div class="col-12">
-                                <HeartbeatBar size="small" :monitor-id="item.id" />
-                            </div>
-                        </div>
-                    </router-link>
-                    
-                </div>
+                <MonitorList />
             </div>
+
             <div class="col-12 col-md-7 col-xl-8">
                 <router-view />
             </div>
@@ -42,119 +17,21 @@
 
 <script>
 
-import HeartbeatBar from "../components/HeartbeatBar.vue";
-import Uptime from "../components/Uptime.vue";
+import MonitorList from "../components/MonitorList.vue";
 
 export default {
     components: {
-        Uptime,
-        HeartbeatBar,
+        MonitorList,
     },
     data() {
         return {}
     },
-    computed: {
-        sortedMonitorList() {
-            let result = Object.values(this.$root.monitorList);
 
-            result.sort((m1, m2) => {
-
-                if (m1.active !== m2.active) {
-                    if (m1.active === 0) {
-                        return 1;
-                    }
-
-                    if (m2.active === 0) {
-                        return -1;
-                    }
-                }
-
-                if (m1.weight !== m2.weight) {
-                    if (m1.weight > m2.weight) {
-                        return -1;
-                    }
-
-                    if (m1.weight < m2.weight) {
-                        return 1;
-                    }
-                }
-
-                return m1.name.localeCompare(m2.name);
-            })
-
-            return result;
-        },
-        showList() {
-            return ! this.$root.isMobile || this.$root.showListMobile;
-        },
-    },
-    methods: {
-        monitorURL(id) {
-            return "/dashboard/" + id;
-        },
-    },
 }
 </script>
 
 <style lang="scss" scoped>
-@import "../assets/vars.scss";
-
 .container-fluid {
     width: 98%
 }
-
-.list {
-    height: auto;
-    min-height: calc(100vh - 240px);
-
-    .item {
-        display: block;
-        text-decoration: none;
-        padding: 15px 15px 12px 15px;
-        border-radius: 10px;
-        transition: all ease-in-out 0.15s;
-
-        &.disabled {
-            opacity: 0.3;
-        }
-
-        .info {
-            white-space: nowrap;
-            overflow: hidden;
-            text-overflow: ellipsis;
-        }
-
-        &:hover {
-            background-color: $highlight-white;
-        }
-
-        &.active {
-            background-color: #cdf8f4;
-        }
-    }
-}
-
-.small-padding {
-    padding-left: 5px !important;
-    padding-right: 5px !important;
-}
-
-.dark {
-    .list {
-        .item {
-            &:hover {
-                background-color: $dark-bg2;
-            }
-
-            &.active {
-                background-color: $dark-bg2;
-            }
-        }
-    }
-}
-
-.monitorItem {
-    width: 100%;
-}
-
 </style>
diff --git a/src/pages/DashboardHome.vue b/src/pages/DashboardHome.vue
index 969652f31..955358080 100644
--- a/src/pages/DashboardHome.vue
+++ b/src/pages/DashboardHome.vue
@@ -37,7 +37,7 @@
             </div>
         </div>
 
-        <div class="shadow-box" style="margin-top: 25px;">
+        <div class="shadow-box" style="margin-top: 25px;overflow-x: scroll">
             <table class="table table-borderless table-hover">
                 <thead>
                     <tr>