From 4f3a1cac430c98c8b3d89faa1aad75efd40072b7 Mon Sep 17 00:00:00 2001
From: zhaojisen <1301338853@qq.com>
Date: Tue, 11 Jun 2024 18:22:15 +0800
Subject: [PATCH 1/2] style: Optimize the styling of region forms in cloud sync

---
 .../Cloud/Account/components/RegionPanel.vue  | 101 ++++++++++++------
 1 file changed, 70 insertions(+), 31 deletions(-)

diff --git a/src/views/assets/Cloud/Account/components/RegionPanel.vue b/src/views/assets/Cloud/Account/components/RegionPanel.vue
index 32d42797b..d4d9b962d 100644
--- a/src/views/assets/Cloud/Account/components/RegionPanel.vue
+++ b/src/views/assets/Cloud/Account/components/RegionPanel.vue
@@ -11,28 +11,40 @@
       :title="$tc('Region')"
       :visible.sync="regionVisible"
       :show-cancel="false"
-      width="'60'"
+      width="60%"
       @confirm="regionVisible=false"
     >
-      <el-row :gutter="12">
-        <el-col v-for="r in allRegions" :key="r.id" :span="6">
-          <el-card
-            shadow="hover"
-            class="region-card"
-            :class="regions.indexOf(r.id) !== -1 ? 'active': ''"
-            :body-style="{ padding: '6px' }"
-            @click.native="handlerCardClick(r.id)"
-          >
-            <span>{{ r.name }}</span>
-          </el-card>
+      <el-row>
+        <el-col>
+          <el-checkbox v-model="checkAll" @change="handleCheckedAllChange">
+            {{ $t('All') }}
+          </el-checkbox>
         </el-col>
       </el-row>
+      <el-checkbox-group
+        v-model="checkedRegion"
+        @change="handleCheckedRegionChange"
+      >
+        <el-row
+          v-for="r in allRegions"
+          :key="r.id"
+          type="flex"
+        >
+          <el-col>
+            <el-checkbox
+              :label="r.id"
+              :value="r.id"
+            >
+              {{ r.name }}
+            </el-checkbox>
+          </el-col>
+        </el-row>
+      </el-checkbox-group>
     </Dialog>
   </div>
 </template>
 
 <script>
-
 import { Dialog } from '@/components'
 import { encryptAttrsField } from '@/views/assets/Cloud/const'
 
@@ -57,9 +69,17 @@ export default {
   },
   data() {
     return {
-      regionVisible: false,
+      content: '',
+      checkedRegion: [],
       allRegions: [],
-      content: ''
+      checkAll: false,
+      regionVisible: false,
+      isIndeterminate: false
+    }
+  },
+  watch: {
+    checkedRegion() {
+      this.updateCheckedStatus()
     }
   },
   mounted() {
@@ -67,7 +87,7 @@ export default {
   },
   methods: {
     refreshContent() {
-      const count = this.regions.length || this.$t('SelectAll')
+      const count = this.checkedRegion.length || this.$t('SelectAll')
       this.content = `${this.$t('Modify')} [${count}]`
     },
     handlerLinkClick() {
@@ -85,34 +105,53 @@ export default {
       this.$axios[method](url, data).then(resp => {
         this.allRegions = resp?.regions
         this.regionVisible = true
+        this.updateCheckedStatus()
       }).catch(error => {
         this.$message.error(this.$tc('CloudRegionTip' + ' ' + error))
       }).finally(() => {
         this.refreshContent()
       })
     },
-    handlerCardClick(regionId) {
-      const index = this.regions.indexOf(regionId)
-      if (index !== -1) {
-        this.regions.splice(index, 1)
-      } else {
-        this.regions.push(regionId)
-      }
-      this.$emit('input', this.regions)
+    handleCheckedAllChange(val) {
+      this.checkedRegion = val ? this.allRegions.map(region => region.id) : []
+      this.isIndeterminate = false
+    },
+    handleCheckedRegionChange(value) {
+      const checkedCount = value.length
+      this.checkAll = checkedCount === this.allRegions.length
+      this.isIndeterminate = checkedCount > 0 && checkedCount < this.allRegions.length
+
+      const checkedRegion = this.allRegions.filter(item => value.includes(item.id))
+
+      this.$emit('input', checkedRegion)
       this.refreshContent()
+    },
+    updateCheckedStatus() {
+      const checkedCount = this.checkedRegion.length
+      this.checkAll = checkedCount === this.allRegions.length
+      this.isIndeterminate = checkedCount > 0 && checkedCount < this.allRegions.length
     }
   }
 }
 </script>
 
 <style lang='scss' scoped>
-.region-card {
-  font-size: 12px;
-  text-align: center;
-  margin-bottom: 6px;
+.el-checkbox {
+  margin-bottom: 10px;
 }
-.el-card.active {
-  color: var(--color-primary);
-  border: 1px solid;
+
+.el-checkbox-group {
+  display: flex;
+  flex-wrap: wrap;
+
+  ::v-deep .el-col {
+
+    .el-checkbox {
+      display: flex;
+      align-items: center;
+      width: 250px;
+      height: 25px;
+    }
+  }
 }
 </style>

From 19c71d4ddd59ea5586861944b13c391d6a13ab84 Mon Sep 17 00:00:00 2001
From: zhaojisen <1301338853@qq.com>
Date: Tue, 11 Jun 2024 18:27:23 +0800
Subject: [PATCH 2/2] fix: Fixed activeMenu duplicate issue

---
 src/views/assets/Cloud/Account/components/CreateDialog.vue | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/views/assets/Cloud/Account/components/CreateDialog.vue b/src/views/assets/Cloud/Account/components/CreateDialog.vue
index 01145a1fc..771f4a74d 100644
--- a/src/views/assets/Cloud/Account/components/CreateDialog.vue
+++ b/src/views/assets/Cloud/Account/components/CreateDialog.vue
@@ -67,7 +67,6 @@ export default {
       activeMenuMap: { 0: 'ProviderPanel', 1: 'AuthPanel', 2: 'AssetPanel', 3: 'ResultPanel' },
       activeMenu: 'ProviderPanel',
       selected: '',
-      activeMenu: 'ProviderPanel',
       firstStepDesc: this.$tc('SelectPlatforms'),
       active: 0,
       account: {}