fix(系统设置): 修复用户组页面bug'

frontend/src/assets/style/global.less

@@ -78,6 +78,8 @@ body {
   font-size: 12px;
   color: rgb(var(--color-text-4));
 }
+
+/* 单航文本缩略 */
 .one-line-text {
   @apply overflow-hidden overflow-ellipsis whitespace-nowrap;
 }

frontend/src/components/business/ms-remove-button/MsRemoveButton.vue

@@ -1,14 +1,14 @@
 <template>
   <MsPopconfirm
-    :ok-text="t('common.remove')"
-    :cancel-text="t('common.cancel')"
     type="error"
     :title="props.title"
     :sub-title-tip="props.subTitleTip"
     :loading="props.loading"
+    :visible="currentVisible"
     @confirm="handleOk"
+    @cancel="handleCancel"
   >
-    <MsButton>{{ t('common.remove') }}</MsButton>
+    <MsButton @click="showPopover">{{ t('common.remove') }}</MsButton>
   </MsPopconfirm>
 </template>
 
@@ -16,6 +16,7 @@
   import { useI18n } from '@/hooks/useI18n';
   import MsButton from '@/components/pure/ms-button/index.vue';
   import MsPopconfirm from '@/components/pure/ms-popconfirm/index.vue';
+  import { ref } from 'vue';
 
   const props = defineProps<{
     title: string;
@@ -27,9 +28,17 @@
     (e: 'ok'): void;
   }>();
 
+  const { t } = useI18n();
+  const currentVisible = ref(false);
+
   const handleOk = () => {
     emit('ok');
   };
+  const handleCancel = () => {
+    currentVisible.value = false;
+  };
 
-  const { t } = useI18n();
+  const showPopover = () => {
+    currentVisible.value = true;
+  };
 </script>

frontend/src/components/pure/ms-popconfirm/index.vue

@@ -1,38 +1,48 @@
 <template>
-  <a-popconfirm
-    v-bind="attrs"
-    :type="props.type"
-    class="w-[352px]"
-    :ok-loading="props.loading"
-    @before-ok="handleConfirm"
-  >
-    <template v-if="props.type === 'error'" #icon>
-      <MsIcon type="icon-icon_warning_filled" class="mr-[2px] text-xl text-[rgb(var(--danger-6))]" />
-    </template>
-    <slot v-if="props.type !== 'error'" name="icon"></slot>
+  <a-popover v-bind="attrs" :type="props.type" :popup-visible="currentVisible" class="w-[352px]" trigger="click">
     <template #content>
-      <span class="font-semibold">
-        {{ props.title }}
-      </span>
-      <div class="py-2 text-sm leading-6 text-[var(--color-text-2)]">
+      <div class="flex flex-row flex-nowrap items-center">
+        <slot name="icon">
+          <MsIcon type="icon-icon_warning_filled" class="mr-[2px] text-xl text-[rgb(var(--danger-6))]" />
+        </slot>
+        <span class="ml-2 font-semibold">
+          {{ props.title }}
+        </span>
+      </div>
+      <div class="ml-8 mt-2 text-sm text-[var(--color-text-2)]">
         {{ props.subTitleTip }}
       </div>
+      <div class="mt-4 flex flex-row flex-nowrap justify-end gap-2">
+        <a-button type="secondary" size="mini" :disabled="props.loading" @click="handleCancel">
+          {{ props.cancelText || t('common.cancel') }}
+        </a-button>
+        <a-button type="primary" size="mini" :loading="props.loading" @click="handleConfirm">
+          {{ props.okText || t('common.remove') }}
+        </a-button>
+      </div>
     </template>
+
     <slot></slot>
-  </a-popconfirm>
+  </a-popover>
 </template>
 
 <script setup lang="ts">
-  import { useAttrs } from 'vue';
+  import { useI18n } from '@/hooks/useI18n';
+  import { ref, useAttrs, watchEffect } from 'vue';
 
   export type types = 'error' | 'info' | 'success' | 'warning';
 
+  const { t } = useI18n();
+
   const props = withDefaults(
     defineProps<{
       title: string;
       subTitleTip: string;
       type: types;
       loading?: boolean;
+      okText?: string;
+      cancelText?: string;
+      visible?: boolean;
     }>(),
     {
       type: 'warning',
@@ -40,11 +50,23 @@
   );
   const emits = defineEmits<{
     (e: 'confirm'): void;
+    (e: 'cancel'): void;
   }>();
+
+  const currentVisible = ref(props.visible || false);
+
   const attrs = useAttrs();
   const handleConfirm = () => {
     emits('confirm');
   };
+
+  const handleCancel = () => {
+    emits('cancel');
+  };
+
+  watchEffect(() => {
+    currentVisible.value = props.visible;
+  });
 </script>
 
 <style scoped lang="less"></style>

frontend/src/views/setting/organization/project/orgProject.vue

@@ -13,7 +13,7 @@
     </div>
     <MsBaseTable v-bind="propsRes" v-on="propsEvent">
       <template #name="{ record }">
-        <span class="overflow-hidden text-ellipsis whitespace-nowrap">{{ record.name }}</span>
+        <span class="one-text-line">{{ record.name }}</span>
         <a-tooltip background-color="#FFFFFF">
           <template #content>
             <span class="text-[var(--color-text-1)]">{{ t('system.project.revokeDeleteToolTip') }}</span>

frontend/src/views/setting/organization/usergroup/components/index.vue

@@ -1,80 +1,78 @@
 <template>
-  <div class="user-group-left">
-    <a-input-search
-      class="w-[252px]"
-      :placeholder="t('system.userGroup.searchHolder')"
-      allow-clear
-      @press-enter="enterData"
-      @search="searchData"
-    />
-    <div class="mt-2 flex flex-col">
-      <div class="flex h-[38px] items-center px-[8px] leading-[24px]">
-        <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.global') }}</div>
-      </div>
-      <div>
-        <div
-          v-for="element in globalUserGroupList"
-          :key="element.id"
-          class="flex h-[38px] cursor-pointer items-center px-[8px]"
-          :class="{
-            'bg-[rgb(var(--primary-1))]': element.id === currentId,
-          }"
-          @click="handleListItemClick(element)"
-        >
-          <div class="flex grow flex-row">
-            <div class="leading-[24px] text-[var(--color-text-1)]">
-              <span class="text-[var(--color-text-1)]">{{ element.name }}</span>
-              <span v-if="element.type" class="text-[var(--color-text-4)]"
-                >（{{ t(`system.userGroup.${element.type}`) }}）</span
-              >
-            </div>
-          </div>
+  <a-input-search
+    class="w-[252px]"
+    :placeholder="t('system.userGroup.searchHolder')"
+    allow-clear
+    @press-enter="enterData"
+    @search="searchData"
+  />
+  <div class="mt-2 flex flex-col">
+    <div class="flex h-[38px] items-center px-[8px] leading-[24px]">
+      <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.global') }}</div>
+    </div>
+    <div>
+      <div
+        v-for="element in globalUserGroupList"
+        :key="element.id"
+        class="flex h-[38px] cursor-pointer items-center px-[8px]"
+        :class="{
+          'bg-[rgb(var(--primary-1))]': element.id === currentId,
+        }"
+        @click="handleListItemClick(element)"
+      >
+        <div class="flex flex-row flex-nowrap">
+          <div class="one-line-text max-w-[156px] text-[var(--color-text-1)]">{{ element.name }}</div>
+          <div v-if="element.type" class="text-[var(--color-text-4)]"
+            >（{{ t(`system.userGroup.${element.type}`) }}）</div
+          >
         </div>
       </div>
     </div>
-    <a-divider class="mt-2" />
-    <div class="mt-2 flex flex-col">
-      <AddOrUpdateUserGroupPopup
-        :visible="addUserGroupVisible"
-        :list="customUserGroupList"
-        @cancel="handleAddUserGroupCancel"
-        @search="initData"
+  </div>
+  <a-divider class="mt-2" />
+  <div class="mt-2 flex flex-col">
+    <AddOrUpdateUserGroupPopup
+      :visible="addUserGroupVisible"
+      :list="customUserGroupList"
+      @cancel="handleAddUserGroupCancel"
+      @search="initData"
+    >
+      <div class="flex h-[38px] items-center justify-between px-[8px] leading-[24px]">
+        <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.custom') }}</div>
+        <div class="cursor-pointer text-[rgb(var(--primary-5))]"
+          ><icon-plus-circle-fill style="font-size: 20px" @click="addUserGroup"
+        /></div>
+      </div>
+    </AddOrUpdateUserGroupPopup>
+    <div>
+      <div
+        v-for="element in customUserGroupList"
+        :key="element.id"
+        class="flex h-[38px] cursor-pointer items-center"
+        :class="{ 'bg-[rgb(var(--primary-1))]': element.id === currentId }"
+        @click="handleListItemClick(element)"
       >
-        <div class="flex h-[38px] items-center justify-between px-[8px] leading-[24px]">
-          <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.custom') }}</div>
-          <div class="cursor-pointer text-[rgb(var(--primary-5))]"
-            ><icon-plus-circle-fill style="font-size: 20px" @click="addUserGroup"
-          /></div>
-        </div>
-      </AddOrUpdateUserGroupPopup>
-      <div>
-        <div
-          v-for="element in customUserGroupList"
-          :key="element.id"
-          class="flex h-[38px] cursor-pointer items-center px-[8px]"
-          :class="{ 'bg-[rgb(var(--primary-1))]': element.id === currentId }"
-          @click="handleListItemClick(element)"
+        <AddOrUpdateUserGroupPopup
+          :id="element.id"
+          :visible="popVisible[element.id]"
+          :default-name="popDefaultName"
+          :list="customUserGroupList"
+          @cancel="() => handlePopConfirmCancel(element.id)"
         >
-          <AddOrUpdateUserGroupPopup
-            :id="element.id"
-            :visible="popVisible[element.id]"
-            :default-name="popDefaultName"
-            :list="customUserGroupList"
-            @cancel="() => handlePopConfirmCancel(element.id)"
-          >
-            <div class="flex grow flex-row justify-between">
-              <div class="leading-[24px] text-[var(--color-text-1)]">
-                <span class="text-[var(--color-text-1)]">{{ element.name }}</span>
-                <span v-if="element.type" class="text-[var(--color-text-4)]"
-                  >（{{ t(`system.userGroup.${element.type}`) }}）</span
+          <div class="flex grow flex-row justify-between px-[8px]">
+            <a-tooltip :content="element.name">
+              <div class="flex flex-row flex-nowrap">
+                <div class="one-line-text max-w-[156px] text-[var(--color-text-1)]">{{ element.name }}</div>
+                <div v-if="element.type" class="text-[var(--color-text-4)]"
+                  >（{{ t(`system.userGroup.${element.type}`) }}）</div
                 >
               </div>
-              <div v-if="element.id === currentId && !element.internal">
-                <MsTableMoreAction :list="customAction" @select="(value) => handleMoreAction(value, element.id)" />
-              </div>
+            </a-tooltip>
+            <div v-if="element.id === currentId && !element.internal">
+              <MsTableMoreAction :list="customAction" @select="(value) => handleMoreAction(value, element.id)" />
             </div>
-          </AddOrUpdateUserGroupPopup>
-        </div>
+          </div>
+        </AddOrUpdateUserGroupPopup>
       </div>
     </div>
   </div>

frontend/src/views/setting/organization/usergroup/orgUserGroup.vue

@@ -10,7 +10,9 @@
       </div>
       <div class="relative w-[100%] overflow-x-scroll p-[24px]">
         <div class="flex flex-row items-center justify-between">
-          <div class="title">{{ store.userGroupInfo.currentName }}</div>
+          <a-tooltip :content="store.userGroupInfo.currentName">
+            <div class="one-line-text max-w-[300px]">{{ store.userGroupInfo.currentName }}</div>
+          </a-tooltip>
           <div class="flex items-center">
             <a-input-search
               v-if="currentTable === 'user'"

frontend/src/views/setting/system/organizationAndProject/components/systemOrganization.vue

@@ -1,7 +1,7 @@
 <template>
   <MsBaseTable v-bind="propsRes" v-on="propsEvent">
     <template #name="{ record }">
-      <span class="overflow-hidden text-ellipsis whitespace-nowrap">{{ record.name }}</span>
+      <span class="one-text-line">{{ record.name }}</span>
       <a-tooltip background-color="#FFFFFF">
         <template #content>
           <span class="text-[var(--color-text-1)]">{{ t('system.organization.revokeDeleteToolTip') }}</span>

frontend/src/views/setting/system/organizationAndProject/components/systemProject.vue

@@ -1,7 +1,7 @@
 <template>
   <MsBaseTable v-bind="propsRes" v-on="propsEvent">
     <template #name="{ record }">
-      <span class="overflow-hidden text-ellipsis whitespace-nowrap">{{ record.name }}</span>
+      <span class="one-text-line">{{ record.name }}</span>
       <a-tooltip background-color="#FFFFFF">
         <template #content>
           <span class="text-[var(--color-text-1)]">{{ t('system.project.revokeDeleteToolTip') }}</span>

frontend/src/views/setting/system/organizationAndProject/index.vue

@@ -53,6 +53,18 @@
   const projectTabeRef = ref();
   const projectVisible = ref(false);
 
+  // 初始化项目数量和组织数量
+  const initOrgAndProjectCount = async () => {
+    try {
+      const res = await getOrgAndProjectCount();
+      organizationCount.value = res.organizationTotal;
+      projectCount.value = res.projectTotal;
+    } catch (error) {
+      // eslint-disable-next-line no-console
+      console.error(error);
+    }
+  };
+
   const tableSearch = () => {
     if (currentTable.value === 'organization') {
       if (orgTableRef.value) {
@@ -69,6 +81,7 @@
         projectTabeRef.value?.fetchData();
       });
     }
+    initOrgAndProjectCount();
   };
 
   const handleSearch = (value: string) => {
@@ -95,18 +108,6 @@
     organizationVisible.value = false;
   };
 
-  // 初始化项目数量和组织数量
-  const initOrgAndProjectCount = async () => {
-    try {
-      const res = await getOrgAndProjectCount();
-      organizationCount.value = res.organizationTotal;
-      projectCount.value = res.projectTotal;
-    } catch (error) {
-      // eslint-disable-next-line no-console
-      console.error(error);
-    }
-  };
-
   watch(
     () => currentTable.value,
     () => {

frontend/src/views/setting/system/usergroup/components/addUserGroupModal.vue

@@ -85,6 +85,8 @@
     currentVisible.value = props.visible;
   });
   const handleCancel = (shouldSearch: boolean) => {
+    form.name = '';
+    form.type = '';
     emit('cancel', shouldSearch);
   };
 

frontend/src/views/setting/system/usergroup/components/index.vue

@@ -1,49 +1,49 @@
 <template>
-  <div class="user-group-left">
-    <a-input-search
-      allow-clear
-      class="w-[252px]"
-      :placeholder="t('system.userGroup.searchHolder')"
-      @press-enter="enterData"
-      @search="searchData"
-    />
-    <div class="mt-2 flex flex-col">
-      <div class="flex h-[38px] items-center justify-between px-[8px] leading-[24px]">
-        <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.global') }}</div>
-        <div class="cursor-pointer text-[rgb(var(--primary-5))]"
-          ><icon-plus-circle-fill style="font-size: 20px" @click="addUserGroup"
-        /></div>
-      </div>
-      <div>
-        <div
-          v-for="element in userGroupList"
-          :key="element.id"
-          class="flex h-[38px] cursor-pointer items-center px-[8px]"
-          :class="{ 'bg-[rgb(var(--primary-1))]': element.id === currentId }"
-          @click="handleListItemClick(element)"
+  <a-input-search
+    allow-clear
+    class="w-[252px]"
+    :placeholder="t('system.userGroup.searchHolder')"
+    @press-enter="enterData"
+    @search="searchData"
+  />
+  <div class="mt-2 flex flex-col">
+    <div class="flex h-[38px] items-center justify-between px-[8px] leading-[24px]">
+      <div class="text-[var(--color-text-input-border)]"> {{ t('system.userGroup.global') }}</div>
+      <div class="cursor-pointer text-[rgb(var(--primary-5))]"
+        ><icon-plus-circle-fill style="font-size: 20px" @click="addUserGroup"
+      /></div>
+    </div>
+    <div>
+      <div
+        v-for="element in userGroupList"
+        :key="element.id"
+        class="flex h-[38px] cursor-pointer items-center"
+        :class="{ 'bg-[rgb(var(--primary-1))]': element.id === currentId }"
+        @click="handleListItemClick(element)"
+      >
+        <popconfirm
+          :visible="popVisible[element.id]"
+          :loading="popLoading[element.id]"
+          :type="popType"
+          :default-name="popDefaultName"
+          :list="userGroupList"
+          @cancel="() => handlePopConfirmCancel(element.id)"
+          @submit="(value: CustomMoreActionItem) => handlePopConfirmSubmit(value,element.id)"
         >
-          <popconfirm
-            :visible="popVisible[element.id]"
-            :loading="popLoading[element.id]"
-            :type="popType"
-            :default-name="popDefaultName"
-            :list="userGroupList"
-            @cancel="() => handlePopConfirmCancel(element.id)"
-            @submit="(value: CustomMoreActionItem) => handlePopConfirmSubmit(value,element.id)"
-          >
-            <div class="draglist-item flex grow flex-row justify-between">
-              <div class="leading-[24px] text-[var(--color-text-1)]">
-                <span class="text-[var(--color-text-1)]">{{ element.name }}</span>
-                <span v-if="element.type" class="text-[var(--color-text-4)]"
-                  >（{{ t(`system.userGroup.${element.type}`) }}）</span
+          <div class="flex grow flex-row justify-between px-[8px]">
+            <a-tooltip :content="element.name">
+              <div class="flex flex-row flex-nowrap">
+                <div class="one-line-text max-w-[156px] text-[var(--color-text-1)]">{{ element.name }}</div>
+                <div v-if="element.type" class="text-[var(--color-text-4)]"
+                  >（{{ t(`system.userGroup.${element.type}`) }}）</div
                 >
               </div>
-              <div v-if="element.id === currentId && !element.internal">
-                <MsTableMoreAction :list="customAction" @select="(value) => handleMoreAction(value, element.id)" />
-              </div>
+            </a-tooltip>
+            <div v-if="element.id === currentId && !element.internal">
+              <MsTableMoreAction :list="customAction" @select="(value) => handleMoreAction(value, element.id)" />
             </div>
-          </popconfirm>
-        </div>
+          </div>
+        </popconfirm>
       </div>
     </div>
   </div>

frontend/src/views/setting/system/usergroup/components/popconfirm.vue

@@ -18,7 +18,7 @@
           <div class="title">{{ message.title }}</div>
         </a-form-item>
         <a-form-item field="name" :rules="[{ validator: validateName }]">
-          <a-input v-if="props.type === 'rename'" v-model="form.name" class="w-[245px]" />
+          <a-input v-if="props.type === 'rename'" v-model="form.name" class="w-[234px]" />
           <a-select v-else v-model="form.name" class="w-[176px]">
             <a-option value="SYSTEM">{{ t('system.userGroup.SYSTEM') }}</a-option>
             <a-option value="ORGANIZATION">{{ t('system.userGroup.ORGANIZATION') }}</a-option>

frontend/src/views/setting/system/usergroup/index.vue

@@ -10,7 +10,9 @@
       </div>
       <div class="relative w-[100%] overflow-x-scroll p-[24px]">
         <div class="flex flex-row items-center justify-between">
-          <div class="title">{{ store.userGroupInfo.currentName }}</div>
+          <a-tooltip :content="store.userGroupInfo.currentName">
+            <div class="one-line-text max-w-[300px]">{{ store.userGroupInfo.currentName }}</div>
+          </a-tooltip>
           <div class="flex items-center">
             <a-input-search
               v-if="currentTable === 'user'"
@@ -127,14 +129,10 @@
 </script>
 
 <style lang="scss" scoped>
-  .title {
-    color: var(--color-text-1);
-  }
-  .user-group {
-    height: calc(100vh - 72px);
-  }
   .user-group-left {
     position: relative;
+    width: 300px;
+    height: calc(100vh - 125px);
     border-right: 1px solid var(--color-border);
     .usergroup-collapse {
       position: absolute;