bfg-common 1.6.30 → 1.6.32

package/components/common/vm/actions/common/select/storage/new/lib/config/table.ts

@@ -9,9 +9,7 @@ import type { UI_I_StatusData } from '~/components/common/vm/actions/common/sele
 import { datastoreLocalizationByState } from '~/components/common/lib/config/states'
 import { getStatus } from '~/components/common/vm/actions/common/select/storage/new/lib/utils/utils'
 
-export const tableDataFunc = (
-  localization: UI_I_Localization
-): UI_I_DataTable => ({
+export const tableDataFunc = (localization: UI_I_Localization): UI_I_DataTable => ({
   id: 'select-storage',
   header: [
     {
@@ -31,7 +29,7 @@ export const tableDataFunc = (
       text: localization.inventorySummary.state,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '160px',
       show: true,
       filter: true,
@@ -42,7 +40,7 @@ export const tableDataFunc = (
       text: localization.common.capacity,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -53,7 +51,7 @@ export const tableDataFunc = (
       text: localization.common.provisioned,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -64,7 +62,7 @@ export const tableDataFunc = (
       text: localization.common.free,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -75,7 +73,7 @@ export const tableDataFunc = (
       text: localization.common.used,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -86,7 +84,7 @@ export const tableDataFunc = (
       text: localization.common.type,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '120px',
       show: true,
       filter: true,
@@ -97,7 +95,7 @@ export const tableDataFunc = (
       text: localization.common.thinProvisioning,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '160px',
       show: true,
       filter: true,
@@ -108,7 +106,7 @@ export const tableDataFunc = (
       text: localization.common.access,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '120px',
       show: true,
       filter: true,
@@ -119,7 +117,7 @@ export const tableDataFunc = (
       text: localization.common.hardwareAcceleration,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '200px',
       show: true,
       filter: true,
@@ -130,7 +128,7 @@ export const tableDataFunc = (
       text: localization.common.driverType,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -141,7 +139,7 @@ export const tableDataFunc = (
       text: localization.common.device,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '150px',
       show: true,
       filter: true,
@@ -152,7 +150,7 @@ export const tableDataFunc = (
       text: localization.common.storageIoControl,
       isSortable: false,
       sort: 'asc',
-      sortColumn: true,
+      sortColumn: false,
       width: '180px',
       show: true,
       filter: true,

package/components/common/vm/actions/confirm/Confirm.vue

@@ -4,6 +4,7 @@
     :sub-title="props.subTitle"
     :description="props.description"
     :modal-texts="props.modalTexts"
+    :loading="props.isLoading"
     @hide-modal="emits('hide')"
     @confirm="emits('confirm')"
   >
@@ -22,9 +23,11 @@ const props = withDefaults(
     subTitle: string
     description: string
     modalTexts?: UI_I_ModalTexts
+    isLoading?: boolean
   }>(),
   {
     modalTexts: undefined,
+    isLoading: false
   }
 )
 

package/components/common/vm/snapshots/DetailView.vue

@@ -7,13 +7,13 @@
     >
       <tbody>
         <tr>
-          <td class="left">{{ localization.common.name }}</td>
+          <td class="left" data-id="vm-snapshot-name-label">{{ localization.common.name }}</td>
           <td class="left snapshot-name-label" data-id="vm-snapshot-name">
             {{ detailData.name }}
           </td>
         </tr>
         <tr>
-          <td class="left">{{ localization.common.description }}</td>
+          <td class="left" data-id="vm-snapshot-description-label">{{ localization.common.description }}</td>
           <td
             class="left snapshot-name-label"
             data-id="vm-snapshot-description"
@@ -22,17 +22,17 @@
           </td>
         </tr>
         <tr>
-          <td class="left">{{ localization.snapshots.timestamp }}</td>
+          <td class="left" data-id="vm-snapshot-timestamp-label">{{ localization.snapshots.timestamp }}</td>
           <td class="left snapshot-name-label" data-id="vm-snapshot-timestamp">
             {{ detailData.timestamp }}
           </td>
         </tr>
         <tr>
-          <td class="left">{{ localization.common.size }}</td>
+          <td class="left" data-id="vm-snapshot-size-label">{{ localization.common.size }}</td>
           <td class="left snapshot-name-label">{{ detailData.size }}</td>
         </tr>
         <tr>
-          <td class="left">
+          <td class="left" data-id="vm-snapshot-of-memory-label">
             {{ localization.common.snapshotTheVirtualMachinesMemory }}
           </td>
           <td class="left snapshot-name-label" data-id="vm-snapshot-of-memory">
@@ -40,7 +40,7 @@
           </td>
         </tr>
         <tr>
-          <td class="left">
+          <td class="left" data-id="vm-snapshot-file-system-label">
             {{ localization.common.quiesceGuestFileSystem }}
           </td>
           <td

package/components/common/wizards/datastore/add/New.vue

@@ -256,4 +256,10 @@ const onCreateDatastore = (): void => emits('submit')
     padding: 10px 12px;
   }
 }
+:deep(.ui-wizard-content-right-container) {
+  padding: 0 30px;
+}
+:deep(.ui-wizard-subtitle-container) + *, :deep(.ui-wizard-subtitle-container) {
+  padding-left: 2px;
+}
 </style>

package/components/common/wizards/datastore/add/steps/nameAndDevice/NameAndDeviceNew.vue

@@ -179,7 +179,7 @@ const onSelectHost = (event: string): void => {
   display: flex;
   flex-direction: column;
   row-gap: 16px;
-  margin: 16px 0 16px 2px;
+  margin: 16px 0;
 
   &__alert {
     align-items: flex-start;

package/components/common/wizards/datastore/add/steps/nameAndDevice/advancedOptions/New.vue

@@ -3,7 +3,7 @@
     <ui-button
       variant="text"
       test-id="advanced-options-button"
-      class="advanced-options-btn btn btn-link"
+      class="advanced-options-btn"
       @click="isShowAdvancedOptions = !isShowAdvancedOptions"
     >
       <template #icon>

package/components/common/wizards/network/add/Add.vue

@@ -618,9 +618,20 @@ const finalValidationFunc = async (
     const stepId = wizard.getStepIdByFieldName(fieldName)
 
     if (stepId !== -1) {
+      let message = ''
+
+      if (validationFields[fieldName]) {
+        message = `${validationFields[
+          fieldName
+        ][0].toUpperCase()}${validationFields[fieldName].substring(1)}`
+
+        message =
+          message.charAt(message.length - 1) !== '.' ? message + '.' : message
+      }
+
       stepHasError = wizard.setValidation(stepId, fieldName, {
-        fieldMessage: validationFields[fieldName],
-        alertMessage: validationFields[fieldName],
+        fieldMessage: message,
+        alertMessage: message,
       })
     }
   }

package/components/common/wizards/network/add/AddNew.vue

@@ -44,22 +44,21 @@
         >
           <template #subTitle>
             <div ref="selectTargetDeviceBlock">
-              <div class="subtitle-block flex flex-col">
+              <div class="alert-block flex flex-col">
                 <template
                   v-for="(message, field, key) in selectedStep.fields"
                   :key="key"
                 >
                   <ui-alert
+                    v-if="!!message.alert"
                     :messages="[message.alert]"
                     test-id="select-target-device-alert"
-                    :class="{
-                      'mb-4':
-                        props.alertMessages[selectedStep.id].length - 1 === key,
-                    }"
                     size="md"
                     @hide="emits('hide-alert', selectedStep.id, field)"
                   />
                 </template>
+              </div>
+              <div class="subtitle-block flex flex-col">
                 <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
                 <ui-line margin="12px 0 0" />
               </div>
@@ -89,22 +88,21 @@
         >
           <template #subTitle>
             <div ref="createStandardSwitchBlock">
-              <div class="subtitle-block flex flex-col">
+              <div class="alert-block flex flex-col">
                 <template
                   v-for="(message, field, key) in selectedStep.fields"
                   :key="key"
                 >
                   <ui-alert
+                    v-if="!!message.alert"
                     :messages="[message.alert]"
                     test-id="create-standard-switch-alert"
-                    :class="{
-                      'mb-4':
-                        props.alertMessages[selectedStep.id].length - 1 === key,
-                    }"
                     size="md"
                     @hide="emits('hide-alert', selectedStep.id, field)"
                   />
                 </template>
+              </div>
+              <div class="subtitle-block flex flex-col">
                 <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
                 <ui-line margin="12px 0 0" />
               </div>
@@ -146,22 +144,21 @@
         >
           <template #subTitle>
             <div ref="portPropertiesBlock">
-              <div class="subtitle-block flex flex-col">
+              <div class="alert-block flex flex-col">
                 <template
                   v-for="(message, field, key) in selectedStep.fields"
                   :key="key"
                 >
                   <ui-alert
+                    v-if="!!message.alert"
                     :messages="[message.alert]"
                     test-id="port-properties-alert"
-                    :class="{
-                      'mb-4':
-                        props.alertMessages[selectedStep.id].length - 1 === key,
-                    }"
                     size="md"
                     @hide="emits('hide-alert', selectedStep.id, field)"
                   />
                 </template>
+              </div>
+              <div class="subtitle-block flex flex-col">
                 <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
                 <ui-line margin="12px 0 0" />
               </div>
@@ -190,22 +187,21 @@
         >
           <template #subTitle>
             <div ref="ipFourSettingsBlock">
-              <div class="subtitle-block flex flex-col">
+              <div class="alert-block flex flex-col">
                 <template
                   v-for="(message, field, key) in selectedStep.fields"
                   :key="key"
                 >
                   <ui-alert
+                    v-if="!!message.alert"
                     :messages="[message.alert]"
                     test-id="ip-four-settings-alert"
-                    :class="{
-                      'mb-4':
-                        props.alertMessages[selectedStep.id].length - 1 === key,
-                    }"
                     size="md"
                     @hide="emits('hide-alert', selectedStep.id, field)"
                   />
                 </template>
+              </div>
+              <div class="subtitle-block flex flex-col">
                 <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
                 <ui-line margin="12px 0 0" />
               </div>
@@ -233,22 +229,21 @@
         >
           <template #subTitle>
             <div ref="connectionSettingsBlock">
-              <div class="subtitle-block flex flex-col">
+              <div class="alert-block flex flex-col">
                 <template
                   v-for="(message, field, key) in selectedStep.fields"
                   :key="key"
                 >
                   <ui-alert
+                    v-if="!!message.alert"
                     :messages="[message.alert]"
                     test-id="connection-settings-alert"
-                    :class="{
-                      'mb-4':
-                        props.alertMessages[selectedStep.id].length - 1 === key,
-                    }"
                     size="md"
                     @hide="emits('hide-alert', selectedStep.id, field)"
                   />
                 </template>
+              </div>
+              <div class="subtitle-block flex flex-col">
                 <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
                 <ui-line margin="12px 0 0" />
               </div>
@@ -463,6 +458,7 @@ const onChangeSteps = async (value: UI_I_WizardStep[]): Promise<void> => {
   }
   :deep(.ui-wizard-content-with-subtitle) {
     padding-right: 0;
+    scrollbar-gutter: unset;
   }
 }
 
@@ -477,7 +473,7 @@ const onChangeSteps = async (value: UI_I_WizardStep[]): Promise<void> => {
 :deep(.ui-alert-item) {
   margin-bottom: 12px;
 
-  &.mb-4 {
+  &:last-of-type {
     margin-bottom: 16px;
   }
 }

package/components/common/wizards/network/add/lib/config/initialData.ts

@@ -37,7 +37,7 @@ export const portPropertiesInitial = (
   localization: UI_I_Localization
 ): UI_I_PortProperties => ({
   networkLabel: '',
-  vlanId: `${localization.common.none} (0)`,
+  vlanId: `${localization.common.none2} (0)`,
   mtu: {
     option: '0',
     size: 1500,
@@ -62,15 +62,15 @@ export const connectionSettingsInitial = (
   localization: UI_I_Localization
 ): UI_I_ConnectionSettings => ({
   networkLabel: 'VM Network',
-  vlanId: `${localization.common.none} (0)`,
+  vlanId: `${localization.common.none2} (0)`,
 })
 
 export const vlanIdSelectDataFunc = (
   localization: UI_I_Localization
 ): UI_I_VlanIdData[] => [
   {
-    text: `${localization.common.none} (0)`,
-    value: `${localization.common.none} (0)`,
+    text: `${localization.common.none2} (0)`,
+    value: `${localization.common.none2} (0)`,
   },
   {
     text: `${localization.common.all} (4095)`,

package/components/common/wizards/network/add/lib/utils/utils.ts

@@ -5,5 +5,5 @@ export const getVlanId = (
   localization: UI_I_Localization
 ): string =>
   (vlanId === `${localization.common.all} (4095)` && '4095') ||
-  (vlanId === `${localization.common.none} (0)` && '0') ||
+  (vlanId === `${localization.common.none2} (0)` && '0') ||
   vlanId

package/components/common/wizards/network/add/steps/ipFourSettings/IpFourSettingsNew.vue

@@ -281,14 +281,8 @@ const fields = computed({
   }
 }
 :deep(.ui-checkbox-label) {
-  text-overflow: ellipsis;
-  white-space: nowrap;
-  overflow: hidden;
-
-  .ui-checkbox-label-text {
-    text-overflow: ellipsis;
-    white-space: nowrap;
-    overflow: hidden;
+  &.md {
+    height: unset;
   }
 }
 .tooltip-text {

package/components/common/wizards/network/add/steps/portProperties/PortPropertiesNew.vue

@@ -299,14 +299,8 @@ const portPropertiesEnabledServices = computed<
   padding: 0 8px 0 12px;
 }
 :deep(.ui-checkbox-label) {
-  text-overflow: ellipsis;
-  white-space: nowrap;
-  overflow: hidden;
-
-  .ui-checkbox-label-text {
-    text-overflow: ellipsis;
-    white-space: nowrap;
-    overflow: hidden;
+  &.md {
+    height: unset;
   }
 }
 </style>

package/components/common/wizards/network/add/steps/selectTargetDevice/SelectTargetDeviceNew.vue

@@ -103,7 +103,7 @@
               <template #row="{ item }">
                 <div v-if="item.iconName" class="browse-btn">
                   <ui-button
-                    test-id="button-1"
+                    test-id="browse-network"
                     type="primary"
                     variant="text"
                     size="lg"
@@ -154,7 +154,7 @@
               <template #row="{ item }">
                 <div v-if="item.iconName" class="browse-btn">
                   <ui-button
-                    test-id="button-1"
+                    test-id="browse-standard-switch"
                     type="primary"
                     variant="text"
                     size="lg"
@@ -203,7 +203,7 @@
               <template #row="{ item }">
                 <div v-if="item.iconName" class="browse-btn">
                   <ui-button
-                    test-id="button-1"
+                    test-id="browse-switch"
                     type="primary"
                     variant="text"
                     size="lg"
@@ -357,10 +357,17 @@ watch(
     :deep(.ui-btn) {
       justify-content: flex-start;
       height: 32px;
+      overflow: hidden;
+      text-overflow: ellipsis;
+      white-space: nowrap;
+      width: 100%;
 
       span {
         text-align: left;
         color: var(--select-text);
+        overflow: hidden;
+        text-overflow: ellipsis;
+        white-space: nowrap;
       }
 
       &:hover {

package/components/common/wizards/network/add/steps/selectTargetDevice/modals/new/SelectNetwork.vue

@@ -46,7 +46,7 @@ import type {
   UI_I_DataTable,
   UI_I_DataTableHeader,
   UI_I_DataTableBody,
-  UI_I_TableTexts
+  UI_I_TableTexts,
 } from '~/node_modules/bfg-uikit/components/ui/dataTable/lib/models/interfaces'
 import type { UI_I_Localization } from '~/lib/models/interfaces'
 import type {
@@ -147,6 +147,10 @@ const onSelectNetwork = (): void => {
 <style scoped lang="scss">
 :deep(.table-container) {
   padding: 8px 32px !important;
+
+  .column-manager-button span {
+    font-weight: 500;
+  }
 }
 
 .row-container {

package/components/common/wizards/network/add/validations/common.ts

@@ -21,7 +21,7 @@ export const checkVlanId = (
   vlanId: string,
   localization: UI_I_Localization
 ): boolean =>
-  vlanId !== `${localization.common.none} (0)` &&
+  vlanId !== `${localization.common.none2} (0)` &&
   vlanId !== `${localization.common.all} (4095)` &&
   (isIntegerNumberInRange(vlanId, 0, 4095) || vlanId.trim() === '')
 

package/components/common/wizards/vm/migrate/steps/computeResource/tableView/new/New.vue

@@ -2,7 +2,7 @@
   <div class="device-table-container">
     <ui-data-table
       :data="data"
-      :options="options"
+      :options="tableConfig.options"
       :loading="props.isLoading"
       :total-pages="props.totalPages"
       :total-items="props.totalItems"
@@ -79,13 +79,10 @@ import type {
   UI_I_TableTexts,
 } from '~/node_modules/bfg-uikit/components/ui/dataTable/models/interfaces'
 import type { UI_I_DataTableSkeleton } from '~/node_modules/bfg-uikit/components/ui/dataTable/models/interfaces'
-import type { UI_T_SelectComputeResourceTabType } from '~/components/common/wizards/vm/migrate/select/storage/lib/models/types'
+import type { UI_T_SelectComputeResourceTabType } from '~/components/common/wizards/vm/migrate/steps/selectStorage/lib/models/types'
 import type { UI_T_ComputeResourceTableItem } from '~/components/common/wizards/vm/migrate/steps/computeResource/lib/models/types'
-import {
-  options,
-  getHeaderDataFunc,
-  getBodyDataFunc,
-} from '~/components/common/wizards/vm/migrate/steps/computeResource/tableView/new/lib/config/hostTable'
+import * as hostTable from '~/components/common/wizards/vm/migrate/steps/computeResource/tableView/new/lib/config/hostTable'
+import * as clusterTable from '~/components/common/wizards/vm/migrate/steps/computeResource/tableView/new/lib/config/clusterTable'
 
 const selectedDataLocal = defineModel<number | null>({ required: true })
 const props = defineProps<{
@@ -120,15 +117,33 @@ const tableSkeleton = ref<UI_I_DataTableSkeleton>({
   withoutPagination: true,
 })
 
+const tableTitleMap = {
+  host: localization.value.common.hosts,
+  cluster: localization.value.common.clusters,
+} as const
+const tableConfigMap = {
+  host: hostTable,
+  cluster: clusterTable,
+} as const
+
+const tableConfig = computed(
+  () => tableConfigMap[props.type as keyof typeof tableConfigMap]
+)
+
 const data = computed<UI_I_DataTable>(() => ({
-  id: 'compute-resource-host-table',
-  title: `${localization.value.common.hosts} (${props.totalItems})`,
+  id: `compute-resource-${props.type}-table`,
+  title: tableDataTitle.value,
   header: deviceHeadItems.value,
   body: deviceBodyItems.value,
 }))
-
+const tableDataTitle = computed<string>(
+  () =>
+    `${tableTitleMap[props.type as keyof typeof tableConfigMap]} (${
+      props.totalItems
+    })`
+)
 const deviceHeadItems = computed<UI_I_DataTableHeader[]>(() =>
-  getHeaderDataFunc(localization.value)
+  tableConfig.value.getHeaderDataFunc(localization.value)
 )
 
 const deviceBodyItems = ref<UI_I_DataTableBody[]>([])
@@ -140,13 +155,16 @@ watch(
       return
     }
 
-    deviceBodyItems.value = getBodyDataFunc(newValue, localization.value)
+    deviceBodyItems.value = tableConfig.value.getBodyDataFunc(
+      newValue,
+      localization.value
+    )
   },
   { deep: true, immediate: true }
 )
 
 const onSelectRow = (value: UI_I_DataTableBody[]): void => {
-  selectedDataLocal.value = value?.[0].row
+  selectedDataLocal.value = value?.[0].row as number
 }
 </script>