From 902f332b195f30f5ac223598fe6f4831580a10bb Mon Sep 17 00:00:00 2001
From: Ramesh Mane <101566080+rameshmane7218@users.noreply.github.com>
Date: Thu, 22 Feb 2024 16:18:14 +0000
Subject: [PATCH] fix: PR review changes #2334

---
 .../nc-gui/components/general/FormBanner.vue    |  8 ++++----
 tests/playwright/pages/Dashboard/Form/index.ts  | 17 -----------------
 .../playwright/tests/db/views/viewForm.spec.ts  | 13 -------------
 3 files changed, 4 insertions(+), 34 deletions(-)

diff --git a/packages/nc-gui/components/general/FormBanner.vue b/packages/nc-gui/components/general/FormBanner.vue
index d34176641f..af362e7493 100644
--- a/packages/nc-gui/components/general/FormBanner.vue
+++ b/packages/nc-gui/components/general/FormBanner.vue
@@ -8,11 +8,11 @@ const { bannerImageUrl } = defineProps<Props>()
 <template>
   <div
     class="w-full max-w-screen-xl mx-auto bg-white border-1 border-gray-200 rounded-3xl overflow-hidden"
-    :style="{
-      ...(bannerImageUrl
+    :style="
+      bannerImageUrl
         ? { 'background-image': `url(${bannerImageUrl})`, 'background-size': 'cover', 'background-position': 'center' }
-        : {}),
-    }"
+        : {}
+    "
   >
     <!-- Todo: aspect ratio and cover image uploader and image cropper to crop image in fixed aspect ratio  -->
     <div v-if="!bannerImageUrl" class="h-full flex items-stretch justify-between">
diff --git a/tests/playwright/pages/Dashboard/Form/index.ts b/tests/playwright/pages/Dashboard/Form/index.ts
index 262b2f22a8..0b23414f2b 100644
--- a/tests/playwright/pages/Dashboard/Form/index.ts
+++ b/tests/playwright/pages/Dashboard/Form/index.ts
@@ -54,22 +54,10 @@ export class FormPage extends BasePage {
     return this.dashboard.get().locator('[data-testid="nc-form-wrapper-submit"]');
   }
 
-  // getFormHiddenColumn() {
-  //   return this.get().locator('[data-testid="nc-form-hidden-column"]');
-  // }
-
   getFormFields() {
     return this.get().locator('[data-testid="nc-form-fields"]');
   }
 
-  // getDragNDropToHide() {
-  //   return this.get().locator('[data-testid="nc-drag-n-drop-to-hide"]');
-  // }
-  //
-  // getFormFieldsRemoveIcon() {
-  //   return this.get().locator('[data-testid="nc-field-remove-icon"]');
-  // }
-
   getFormFieldsRequired() {
     return this.get().locator('[data-testid="nc-form-input-required"] + button');
   }
@@ -126,9 +114,6 @@ export class FormPage extends BasePage {
       const dst = this.get().locator(`[data-testid="nc-drag-n-drop-to-hide"]`);
       await src.dragTo(dst);
     } else if (mode === 'hideField') {
-      // const src = this.get().locator(`.nc-form-drag-${field.replace(' ', '')}`);
-      // await src.locator(`[data-testid="nc-field-remove-icon"]`).click();
-
       // in form-v2, hide field will be using right sidebar
       await this.formFields.locator(`[data-testid="nc-form-field-item-${field}"]`).locator('.nc-switch').click();
     }
@@ -146,8 +131,6 @@ export class FormPage extends BasePage {
       await src.dragTo(dst, { trial: true });
       await src.dragTo(dst);
     } else if (mode === 'clickField') {
-      // const src = this.get().locator(`[data-testid="nc-form-hidden-column-${field}"]`);
-      // await src.click();
       await this.formFields.locator(`[data-testid="nc-form-field-item-${field}"]`).locator('.nc-switch').click();
     }
   }
diff --git a/tests/playwright/tests/db/views/viewForm.spec.ts b/tests/playwright/tests/db/views/viewForm.spec.ts
index 10f675466e..2c1e0f746b 100644
--- a/tests/playwright/tests/db/views/viewForm.spec.ts
+++ b/tests/playwright/tests/db/views/viewForm.spec.ts
@@ -49,19 +49,6 @@ test.describe('Form view', () => {
       fields: ['LastUpdate', 'Country', 'Cities'],
     });
 
-    // drag-drop feature to remove a field will not be supported in form v2
-    // remove & verify (drag-drop)
-    // await form.removeField({ field: 'Cities', mode: 'dragDrop' });
-    // await form.verifyFormViewFieldsOrder({
-    //   fields: ['LastUpdate', 'Country'],
-    // });
-
-    // add & verify (drag-drop)
-    // await form.addField({ field: 'Cities', mode: 'dragDrop' });
-    // await form.verifyFormViewFieldsOrder({
-    //   fields: ['LastUpdate', 'Country', 'Cities'],
-    // });
-
     // remove & verify (hide field button)
     await form.removeField({ field: 'Cities', mode: 'hideField' });
     await form.verifyFormViewFieldsOrder({