Remove drag and drop reordering and work with btns.

Author: sten

resources/views/livewire/menu-tree-item.blade.php

@@ -1,14 +1,6 @@
 <div class="menu-tree-item" data-item-id="{{ $item->id }}" wire:key="tree-item-{{ $item->id }}">
     <div class="flex items-center p-4 bg-white dark:bg-gray-800 rounded-lg border border-gray-200 dark:border-gray-700 group hover:bg-gray-50 dark:hover:bg-gray-700 transition-colors {{ $depth > 0 ? 'ml-' . ($depth * 8) : '' }}">
 
-        @if($showActions)
-        <!-- Drag Handle -->
-        <div class="drag-handle cursor-move text-gray-400 hover:text-gray-600 dark:hover:text-gray-300 mr-4">
-            <svg class="w-5 h-5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-                <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M4 8h16M4 16h16"></path>
-            </svg>
-        </div>
-        @endif
 
         <!-- Expand/Collapse Button for items with children -->
         @if($this->hasChildren())
@@ -54,6 +46,24 @@ class="h-5 w-5 text-gray-500 text-lg"/>
                 @if($showActions)
                 <!-- Actions -->
                 <div class="flex items-center gap-1">
+                    <x-filament::button
+                        color="gray"
+                        size="xs"
+                        icon="heroicon-o-arrow-up"
+                        tooltip="Move up"
+                        label-sr-only="Move up"
+                        wire:click="moveUp"
+                    ></x-filament::button>
+
+                    <x-filament::button
+                        color="gray"
+                        size="xs"
+                        icon="heroicon-o-arrow-down"
+                        tooltip="Move down"
+                        label-sr-only="Move down"
+                        wire:click="moveDown"
+                    ></x-filament::button>
+
                     @if($this->canHaveChildren())
                         <x-filament::button
                             color="gray"

src/Resources/MenuResource/Pages/ManageMenuItems.php

@@ -886,4 +886,84 @@ public function handleShowDeleteModal(array $data): void
             $this->mountAction('deleteMenuItem', ['itemId' => $itemId]);
         }
     }
+
+    #[On('move-item-up')]
+    public function handleMoveItemUp(array $data): void
+    {
+        $itemId = $data['itemId'] ?? null;
+        if ($itemId) {
+            $this->moveItemUp($itemId);
+        }
+    }
+
+    #[On('move-item-down')]
+    public function handleMoveItemDown(array $data): void
+    {
+        $itemId = $data['itemId'] ?? null;
+        if ($itemId) {
+            $this->moveItemDown($itemId);
+        }
+    }
+
+    public function moveItemUp(int $itemId): void
+    {
+        try {
+            $item = $this->getMenuItemSecurely($itemId);
+            if (!$item) {
+                Notification::make()
+                    ->title(flexiblePagesTrans('menu_items.errors.item_not_found'))
+                    ->danger()
+                    ->send();
+                return;
+            }
+
+            // Move up means move to previous sibling
+            $previousSibling = $item->getPrevSibling();
+            if ($previousSibling) {
+                $item->beforeNode($previousSibling)->save();
+                $this->refreshTree();
+                
+                Notification::make()
+                    ->title('Menu item moved up successfully.')
+                    ->success()
+                    ->send();
+            }
+        } catch (Exception $e) {
+            Notification::make()
+                ->title('Failed to move menu item: ' . $e->getMessage())
+                ->danger()
+                ->send();
+        }
+    }
+
+    public function moveItemDown(int $itemId): void
+    {
+        try {
+            $item = $this->getMenuItemSecurely($itemId);
+            if (!$item) {
+                Notification::make()
+                    ->title(flexiblePagesTrans('menu_items.errors.item_not_found'))
+                    ->danger()
+                    ->send();
+                return;
+            }
+
+            // Move down means move to next sibling
+            $nextSibling = $item->getNextSibling();
+            if ($nextSibling) {
+                $item->afterNode($nextSibling)->save();
+                $this->refreshTree();
+                
+                Notification::make()
+                    ->title('Menu item moved down successfully.')
+                    ->success()
+                    ->send();
+            }
+        } catch (Exception $e) {
+            Notification::make()
+                ->title('Failed to move menu item: ' . $e->getMessage())
+                ->danger()
+                ->send();
+        }
+    }
 }