bfg-common 1.6.65 → 1.6.67

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

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

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

@@ -256,10 +256,4 @@ 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/lib/config/steps.ts

@@ -130,7 +130,7 @@ export const stepsFunc = (
   // },
   {
     id: dynamicSteps.readyComplete,
-    title: localization.vmWizard.readyComplete,
+    title: localization.common.readyComplete,
     subTitle: localization.common.readyCompleteDatastoreDesc,
     status: UI_E_WIZARD_STATUS.INACTIVE,
     fields: {},

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;
+  margin: 16px 0 16px 2px;
 
   &__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"
+      class="advanced-options-btn btn btn-link"
       @click="isShowAdvancedOptions = !isShowAdvancedOptions"
     >
       <template #icon>

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

@@ -618,20 +618,9 @@ 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: message,
-        alertMessage: message,
+        fieldMessage: validationFields[fieldName],
+        alertMessage: validationFields[fieldName],
       })
     }
   }

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

@@ -44,21 +44,22 @@
         >
           <template #subTitle>
             <div ref="selectTargetDeviceBlock">
-              <div class="alert-block flex flex-col">
+              <div class="subtitle-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>
@@ -88,21 +89,22 @@
         >
           <template #subTitle>
             <div ref="createStandardSwitchBlock">
-              <div class="alert-block flex flex-col">
+              <div class="subtitle-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>
@@ -144,21 +146,22 @@
         >
           <template #subTitle>
             <div ref="portPropertiesBlock">
-              <div class="alert-block flex flex-col">
+              <div class="subtitle-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>
@@ -187,21 +190,22 @@
         >
           <template #subTitle>
             <div ref="ipFourSettingsBlock">
-              <div class="alert-block flex flex-col">
+              <div class="subtitle-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>
@@ -229,21 +233,22 @@
         >
           <template #subTitle>
             <div ref="connectionSettingsBlock">
-              <div class="alert-block flex flex-col">
+              <div class="subtitle-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>
@@ -458,7 +463,6 @@ const onChangeSteps = async (value: UI_I_WizardStep[]): Promise<void> => {
   }
   :deep(.ui-wizard-content-with-subtitle) {
     padding-right: 0;
-    scrollbar-gutter: unset;
   }
 }
 
@@ -473,7 +477,7 @@ const onChangeSteps = async (value: UI_I_WizardStep[]): Promise<void> => {
 :deep(.ui-alert-item) {
   margin-bottom: 12px;
 
-  &:last-of-type {
+  &.mb-4 {
     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.none2} (0)`,
+  vlanId: `${localization.common.none} (0)`,
   mtu: {
     option: '0',
     size: 1500,
@@ -62,21 +62,19 @@ export const connectionSettingsInitial = (
   localization: UI_I_Localization
 ): UI_I_ConnectionSettings => ({
   networkLabel: 'VM Network',
-  vlanId: `${localization.common.none2} (0)`,
+  vlanId: `${localization.common.none} (0)`,
 })
 
 export const vlanIdSelectDataFunc = (
   localization: UI_I_Localization
 ): UI_I_VlanIdData[] => [
   {
-    text: `${localization.common.none2} (0)`,
-    value: `${localization.common.none2} (0)`,
-    testId: 'vlan-id-start'
+    text: `${localization.common.none} (0)`,
+    value: `${localization.common.none} (0)`,
   },
   {
     text: `${localization.common.all} (4095)`,
     value: `${localization.common.all} (4095)`,
-    testId: 'vlan-id-end'
   },
 ]
 

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

@@ -145,7 +145,7 @@ export const stepsFunc = (
   },
   {
     id: 7,
-    title: localization.vmWizard.readyComplete,
+    title: localization.common.readyComplete,
     subTitle: localization.common.addNetworkingReadyCompleteSubtitle,
     status: UI_E_WIZARD_STATUS.INACTIVE,
     fields: {},

package/components/common/wizards/network/add/lib/models/interfaces.ts

@@ -177,7 +177,6 @@ export interface UI_I_AddNetworkingFourthOrFifthSchemeDataSend {
 export interface UI_I_VlanIdData {
   text: string
   value: string
-  testId?: string
 }
 
 export interface UI_I_NetworkValidation {

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.none2} (0)` && '0') ||
+  (vlanId === `${localization.common.none} (0)` && '0') ||
   vlanId

package/components/common/wizards/network/add/steps/connectionSettings/ConnectionSettingsOld.vue

@@ -18,7 +18,6 @@
               <fieldset>
                 <atoms-tooltip-error
                   :has-error="!!props.messagesFields.name.field"
-                  test-id="network-label-error"
                 >
                   <template #elem>
                     <input
@@ -73,10 +72,7 @@
                         class="is-error tooltip-trigger"
                         name="info"
                       />
-                      <div
-                        class="tooltip-top-left tooltip-content error"
-                        data-id="vlan-id-error"
-                      >
+                      <div class="tooltip-top-left tooltip-content error">
                         {{ props.messagesFields.vlan.field }}
                       </div>
                     </div>

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

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

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

@@ -53,7 +53,6 @@
                     :has-error="
                       !!props.messagesFields['ipv4_settings.ipv4'].field
                     "
-                    test-id="address-error"
                   >
                     <template #elem>
                       <input
@@ -89,7 +88,6 @@
                     :has-error="
                       !!props.messagesFields['ipv4_settings.mask'].field
                     "
-                    test-id="mask-error"
                   >
                     <template #elem>
                       <input
@@ -144,7 +142,6 @@
                       :has-error="
                         !!props.messagesFields['ipv4_settings.gateway'].field
                       "
-                      test-id="gateway-error"
                     >
                       <template #elem>
                         <input

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

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

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

@@ -22,7 +22,6 @@
               <fieldset>
                 <atoms-tooltip-error
                   :has-error="!!props.messagesFields.network_name.field"
-                  test-id="network-label-error"
                 >
                   <template #elem>
                     <input
@@ -78,7 +77,7 @@
                         class="is-error tooltip-trigger"
                         name="info"
                       />
-                      <div class="tooltip-top-left tooltip-content error" data-id="vlan-id-error">
+                      <div class="tooltip-top-left tooltip-content error">
                         {{ props.messagesFields['new_network.vlan'].field }}
                       </div>
                     </div>
@@ -112,7 +111,6 @@
               </div>
               <atoms-tooltip-error
                 :has-error="!!props.messagesFields.mtu.field"
-                test-id="invalid-mtu-error"
               >
                 <template #elem>
                   <input
@@ -241,9 +239,7 @@ const emits = defineEmits<{
 }>()
 
 const localization = computed<UI_I_Localization>(() => useLocal())
-const vlanIdSelectDataLocal = computed<UI_I_Localization>(() =>
-  vlanIdSelectDataFunc(localization.value)
-)
+const vlanIdSelectDataLocal = computed<UI_I_Localization>(() => vlanIdSelectDataFunc(localization.value))
 
 const fields = computed({
   get(): UI_I_PortProperties {

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="browse-network"
+                    test-id="button-1"
                     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="browse-standard-switch"
+                    test-id="button-1"
                     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="browse-switch"
+                    test-id="button-1"
                     type="primary"
                     variant="text"
                     size="lg"
@@ -357,17 +357,10 @@ 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/SelectTargetDeviceOld.vue

@@ -34,7 +34,6 @@
                   fieldsLocal.selected === '0' &&
                   !!props.messagesFields.network.field
                 "
-                test-id="existing-network-error"
               >
                 <template #elem>
                   <fieldset class="fieldset-block">
@@ -98,7 +97,6 @@
                   fieldsLocal.selected === '1' &&
                   !!props.messagesFields.switch.field
                 "
-                test-id="existing-standard-switch-error"
               >
                 <template #elem>
                   <fieldset class="fieldset-block">
@@ -162,7 +160,6 @@
                   fieldsLocal.selected === '2' &&
                   !!props.messagesFields.switch2.field
                 "
-                test-id="standard-switch-error"
               >
                 <template #elem>
                   <fieldset class="fieldset-block">
@@ -232,7 +229,6 @@
                       :has-error="
                         !!props.messagesFields['new_switch.mtu'].field
                       "
-                      test-id="invalid-mtu-error"
                     >
                       <template #elem>
                         <input

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,10 +147,6 @@ 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.none2} (0)` &&
+  vlanId !== `${localization.common.none} (0)` &&
   vlanId !== `${localization.common.all} (4095)` &&
   (isIntegerNumberInRange(vlanId, 0, 4095) || vlanId.trim() === '')
 

package/components/common/wizards/vm/migrate/New.vue

@@ -187,13 +187,8 @@
       >
         <template #subTitle>
           <div ref="subTitleBlock6">
-            <div class="subtitle-block flex-row items-start">
+            <div class="subtitle-block">
               <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
-
-              <common-wizards-vm-migrate-vm-origin
-                v-if="props.vmOrigin"
-                :vm-origin="props.vmOrigin"
-              />
             </div>
           </div>
         </template>
@@ -212,13 +207,8 @@
       >
         <template #subTitle>
           <div ref="subTitleBlock7">
-            <div class="subtitle-block flex-row items-start">
+            <div class="subtitle-block">
               <ui-wizard-subtitle :sub-title="selectedStep.subTitle" />
-
-              <common-wizards-vm-migrate-vm-origin
-                v-if="props.vmOrigin"
-                :vm-origin="props.vmOrigin"
-              />
             </div>
           </div>
         </template>
@@ -303,7 +293,7 @@ const texts = computed<UI_I_WizardTexts>(() => ({
   back: localization.value.common.backCap,
   processing: localization.value.common.processing,
   next: localization.value.common.next,
-  finish: localization.value.common.migrate,
+  finish: localization.value.common.create,
   incompleteTitle: localization.value.common.incompleteProcess,
   incompleteMessage: localization.value.common.incompleteProcessMessage,
   incompleteCancel: localization.value.common.cancel,

package/components/common/wizards/vm/migrate/lib/config/constructDataReady.ts

@@ -26,7 +26,6 @@ const migrateOnlyStorageDataView = (
       label: localization.common.storage,
       value: storage?.name,
       permissions: ['batch-configure'],
-      iconClassName: 'vsphere-icon-datastore',
     },
     {
       label: localization.common.diskFormat,
@@ -125,7 +124,6 @@ const storageDetailsFunc = (
       label: localization.common.virtualMachine,
       value: name,
       permissions: ['batch-configure', 'configure-disk'],
-      iconClassName: 'vsphere-icon-vm',
     },
   ]
 
@@ -150,7 +148,6 @@ const storageDetailsFunc = (
           label: localization.common[key],
           value: value.name,
           permissions: [],
-          iconClassName: `vsphere-icon-${key}`,
         })
       }
     })
@@ -168,7 +165,6 @@ const storageDetailsFunc = (
         label: localization.common.host,
         value: '', //selectedComputeResource.selectedNode.name,
         permissions: [],
-        iconClassName: 'vsphere-icon-host',
       },
       {
         label: localization.migrateVm.migrationStrategy,
@@ -180,7 +176,6 @@ const storageDetailsFunc = (
         label: localization.common.networks,
         value: setSelectNetworkDataView(localization, form),
         permissions: [],
-        iconClassName: 'vsphere-icon-network',
       },
     ]
     const storageResult = migrateOnlyStorageDataView(localization, form, name)

package/components/common/wizards/vm/migrate/lib/config/steps.ts

@@ -30,8 +30,7 @@ export const stepsFunc = (
     {
       id: dynamicSteps.selectMigrationType,
       title: localization.common.selectMigrationType,
-      subTitle:
-        localization.migrateVm.changeComputeResourceStorageVirtualMachine,
+      subTitle: localization.migrateVm.changeComputeResourceStorageVirtualMachine,
       status: UI_E_WIZARD_STATUS[!isScheduledTasks ? 'SELECTED' : 'INACTIVE'],
       fields: {},
       isValid: true,
@@ -100,7 +99,7 @@ export const stepsFunc = (
     {
       id: dynamicSteps.readyComplete,
       title: localization.common.readyComplete,
-      subTitle: localization.migrateVm.reviewDetailsBeforeVmMigrated,
+      subTitle: '',
       status: UI_E_WIZARD_STATUS.INACTIVE,
       fields: {},
       isValid: true,

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

@@ -88,9 +88,8 @@ const onTabChange = (name: keyof UI_I_SelectedTableDataKeys): void => {
 @import 'assets/scss/common/mixins';
 .compute-resource {
   @include flex($dir: column);
+  top: 16px;
   height: inherit;
-  margin-top: 16px;
-  margin-bottom: 16px;
   .tree-view-wrap {
     max-height: 250px;
     min-height: 200px;