refactor: small ui changes

Signed-off-by: mertmit <mertmit99@gmail.com>

packages/nc-gui/components/dlg/BulkUpdate.vue

@@ -294,7 +294,7 @@ onMounted(() => {
               <div class="text-gray group absolute top-4 right-12">
                 <component
                   :is="iconMap.eyeSlash"
-                  class="opacity-0 nc-field-remove-icon group-hover:text-red-500 cursor-pointer"
+                  class="opacity-0 nc-field-remove-icon group-hover:text-red-500 cursor-pointer !text-xl"
                   data-testid="nc-bulk-update-fields-remove-icon"
                   @click="handleRemove(element)"
                 />
@@ -383,7 +383,7 @@ onMounted(() => {
           <button
             v-if="fields.length > editColumns.length"
             type="button"
-            class="nc-bulk-update-add-all color-transition bg-white transform hover:(text-primary ring ring-accent ring-opacity-100) active:translate-y-[1px] px-2 py-1 shadow-md rounded"
+            class="nc-bulk-update-add-all color-transition bg-white transform hover:(text-primary ring-1 ring-primary ring-opacity-100) active:translate-y-[1px] px-2 py-1 shadow-md rounded"
             data-testid="nc-bulk-update-add-all"
             tabindex="-1"
             @click="addAllColumns"
@@ -395,7 +395,7 @@ onMounted(() => {
           <button
             v-if="editColumns.length"
             type="button"
-            class="nc-bulk-update-remove-all color-transition bg-white transform hover:(text-primary ring ring-accent ring-opacity-100) active:translate-y-[1px] px-2 py-1 shadow-md rounded"
+            class="nc-bulk-update-remove-all color-transition bg-white transform hover:(text-primary ring-1 ring-primary ring-opacity-100) active:translate-y-[1px] px-2 py-1 shadow-md rounded"
             data-testid="nc-bulk-update-remove-all"
             tabindex="-1"
             @click="removeAllColumns"

packages/nc-gui/components/smartsheet/Grid.vue

@@ -1128,11 +1128,23 @@ function addEmptyRow(row?: number) {
               </div>
             </a-menu-item>
 
+            <a-menu-item
+              v-if="!contextMenuClosing && !contextMenuTarget && data.some((r) => r.rowMeta.selected)"
+              @click="bulkUpdateDlg = true"
+            >
+              <div v-e="['a:row:update-bulk']" class="nc-project-menu-item">
+                <component :is="iconMap.edit" />
+                <!-- TODO i18n -->
+                Update Selected Rows
+              </div>
+            </a-menu-item>
+
             <a-menu-item
               v-if="!contextMenuClosing && !contextMenuTarget && data.some((r) => r.rowMeta.selected)"
               @click="deleteSelectedRows"
             >
-              <div v-e="['a:row:delete-bulk']" class="nc-project-menu-item">
+              <div v-e="['a:row:delete-bulk']" class="nc-project-menu-item text-red-500">
+                <component :is="iconMap.delete" />
                 <!-- Delete Selected Rows -->
                 {{ $t('activity.deleteSelectedRow') }}
               </div>
@@ -1169,16 +1181,6 @@ function addEmptyRow(row?: number) {
                 {{ $t('general.copy') }}
               </div>
             </a-menu-item>
-
-            <a-menu-item
-              v-if="!contextMenuClosing && !contextMenuTarget && data.some((r) => r.rowMeta.selected)"
-              @click="bulkUpdateDlg = true"
-            >
-              <div class="nc-project-menu-item">
-                <!-- TODO i18n -->
-                Update Selected Rows
-              </div>
-            </a-menu-item>
           </a-menu>
         </template>
       </a-dropdown>