Save re-ordering

Author: sten

resources/views/filament/resources/menu-resource/pages/manage-menu-items.blade.php

@@ -72,6 +72,11 @@ function menuItemsManager(config) {
                     this.$wire.on('menu-items-updated', () => {
                         this.refreshMenuItems();
                     });
+                    
+                    // Listen for reorder completion (no need to refresh, just hide loading)
+                    this.$wire.on('menu-items-reordered', () => {
+                        this.loading = false;
+                    });
                 },
 
                 refreshMenuItems() {
@@ -122,10 +127,11 @@ function menuItemsManager(config) {
                     // Call the Livewire method to save the new order
                     this.$wire.reorderMenuItems(items).then(() => {
                         this.loading = false;
+                        // Don't manually refresh - the server-side refreshTree() will handle it
                     }).catch((error) => {
                         console.error('Reorder failed:', error);
                         this.loading = false;
-                        // Revert the visual change by refreshing
+                        // Only refresh on error to revert changes
                         this.refreshMenuItems();
                     });
                 },

src/Resources/MenuResource/Pages/ManageMenuItems.php

@@ -325,7 +325,9 @@ public function reorderMenuItems(array $orderedItems): void
             // Process the reordering with proper nested set operations
             $this->processNestedSetReorder($orderedItems);
 
-            $this->refreshTree();
+            // Don't refresh tree for reordering - the visual order is already correct
+            // Only dispatch event for JavaScript to hide loading state
+            $this->dispatch('menu-items-reordered');
 
             Notification::make()
                 ->title(flexiblePagesTrans('menu_items.messages.items_reordered'))