fix: complete admin workflows and frontend QA issues

Author: mtalavi

app/Filament/Resources/Galleries/GalleryResource.php

@@ -9,6 +9,7 @@
 use App\Filament\Resources\Galleries\Schemas\GalleryForm;
 use App\Filament\Resources\Galleries\Schemas\GalleryInfolist;
 use App\Filament\Resources\Galleries\Tables\GalleriesTable;
+use App\Filament\Support\RequiresPermission;
 use App\Models\Gallery;
 use BackedEnum;
 use Filament\Resources\Resource;
@@ -18,6 +19,8 @@
 
 class GalleryResource extends Resource
 {
+    use RequiresPermission;
+
     protected static ?string $model = Gallery::class;
 
     protected static string|BackedEnum|null $navigationIcon = Heroicon::OutlinedPhoto;

app/Filament/Resources/Galleries/Schemas/GalleryForm.php

@@ -4,6 +4,8 @@
 
 use App\Filament\Support\SecureUpload;
 use App\Filament\Support\SeoForm;
+use App\Models\Gallery;
+use App\Support\SortOrder;
 use Filament\Forms\Components\FileUpload;
 use Filament\Forms\Components\Repeater;
 use Filament\Forms\Components\Select;
@@ -27,9 +29,13 @@ public static function configure(Schema $schema): Schema
                         TextInput::make('slug')->required()->unique(ignoreRecord: true),
                         Select::make('status')->options(['draft' => 'Draft', 'published' => 'Published'])->default('draft')->required(),
                         Toggle::make('is_featured')->label('Featured gallery'),
-                        TextInput::make('sort_order')->numeric()->default(0),
+                        TextInput::make('sort_order')
+                            ->numeric()
+                            ->default(fn (): int => SortOrder::next(Gallery::class))
+                            ->helperText('Used to order galleries. New galleries get the next order automatically.'),
                         SecureUpload::image(FileUpload::make('cover_image'), 'galleries')
                             ->imagePreviewHeight('180')
+                            ->helperText('Cover image. Allowed: JPG, JPEG, PNG, WEBP. Max size: 5 MB. Recommended: 1600x900 or wider.')
                             ->columnSpanFull(),
                     ])
                     ->columns(2),
@@ -42,18 +48,27 @@ public static function configure(Schema $schema): Schema
                     ]),
 
                 Section::make('Images')
-                    ->description('Add multiple images with optional captions for responsive public grids.')
+                    ->description('Add multiple image rows for responsive public grids. Allowed: JPG, JPEG, PNG, WEBP. Max size: 5 MB per image. Recommended: 1200px wide or larger. Server php.ini upload_max_filesize and post_max_size must also allow the selected files.')
                     ->schema([
                         Repeater::make('images')
                             ->schema([
                                 SecureUpload::image(FileUpload::make('path'), 'galleries/items')
-                                    ->imagePreviewHeight('140'),
+                                    ->label('Image file')
+                                    ->imagePreviewHeight('140')
+                                    ->helperText('Upload one image for this row. Add more rows for multiple images.'),
                                 TextInput::make('url')
                                     ->url()
-                                    ->helperText('Optional external image URL.'),
+                                    ->helperText('Optional external image URL. Use either an upload or an external URL.'),
                                 TextInput::make('caption'),
+                                TextInput::make('alt')
+                                    ->label('Alt text')
+                                    ->helperText('Optional accessible image description.'),
+                                TextInput::make('sort_order')
+                                    ->numeric()
+                                    ->helperText('Optional. Rows are auto-ordered when left empty.'),
                             ])
                             ->columns(3)
+                            ->maxItems(50)
                             ->reorderable()
                             ->addActionLabel('Add image')
                             ->columnSpanFull(),

app/Filament/Resources/Galleries/Tables/GalleriesTable.php

@@ -22,6 +22,7 @@ class GalleriesTable
     public static function configure(Table $table): Table
     {
         return TableUx::apply($table
+            ->defaultSort('sort_order')
             ->columns([
                 ImageColumn::make('cover_url')
                     ->label('Cover')

app/Filament/Resources/MediaItems/MediaItemResource.php

@@ -9,6 +9,7 @@
 use App\Filament\Resources\MediaItems\Schemas\MediaItemForm;
 use App\Filament\Resources\MediaItems\Schemas\MediaItemInfolist;
 use App\Filament\Resources\MediaItems\Tables\MediaItemsTable;
+use App\Filament\Support\RequiresPermission;
 use App\Models\MediaItem;
 use BackedEnum;
 use Filament\Resources\Resource;
@@ -18,6 +19,8 @@
 
 class MediaItemResource extends Resource
 {
+    use RequiresPermission;
+
     protected static ?string $model = MediaItem::class;
 
     protected static string|BackedEnum|null $navigationIcon = Heroicon::OutlinedPhoto;

app/Filament/Resources/MediaItems/Schemas/MediaItemForm.php

@@ -3,6 +3,8 @@
 namespace App\Filament\Resources\MediaItems\Schemas;
 
 use App\Filament\Support\SecureUpload;
+use App\Models\MediaItem;
+use App\Support\SortOrder;
 use Filament\Forms\Components\FileUpload;
 use Filament\Forms\Components\Select;
 use Filament\Forms\Components\Textarea;
@@ -28,7 +30,10 @@ public static function configure(Schema $schema): Schema
                             ->required(),
                         Select::make('type')->options(['image' => 'Image', 'document' => 'Document'])->default('image')->required(),
                         Select::make('status')->options(['draft' => 'Draft', 'published' => 'Published'])->default('published')->required(),
-                        TextInput::make('sort_order')->numeric()->default(0),
+                        TextInput::make('sort_order')
+                            ->numeric()
+                            ->default(fn (): int => SortOrder::next(MediaItem::class))
+                            ->helperText('Used to order media in public grids. New media gets the next order automatically.'),
                     ])
                     ->columns(2),
 

app/Filament/Resources/MediaItems/Tables/MediaItemsTable.php

@@ -20,6 +20,7 @@ class MediaItemsTable
     public static function configure(Table $table): Table
     {
         return TableUx::apply($table
+            ->defaultSort('sort_order')
             ->columns([
                 ImageColumn::make('url')
                     ->label('Preview')

app/Filament/Resources/Menus/MenuResource.php

@@ -9,6 +9,7 @@
 use App\Filament\Resources\Menus\Schemas\MenuForm;
 use App\Filament\Resources\Menus\Schemas\MenuInfolist;
 use App\Filament\Resources\Menus\Tables\MenusTable;
+use App\Filament\Support\RequiresPermission;
 use App\Models\Menu;
 use BackedEnum;
 use Filament\Resources\Resource;
@@ -18,6 +19,8 @@
 
 class MenuResource extends Resource
 {
+    use RequiresPermission;
+
     protected static ?string $model = Menu::class;
 
     protected static string|BackedEnum|null $navigationIcon = Heroicon::OutlinedBars3;

app/Filament/Resources/Menus/Schemas/MenuForm.php

@@ -2,6 +2,8 @@
 
 namespace App\Filament\Resources\Menus\Schemas;
 
+use App\Models\Menu;
+use App\Support\SortOrder;
 use Filament\Forms\Components\Repeater;
 use Filament\Forms\Components\Select;
 use Filament\Forms\Components\TextInput;
@@ -16,39 +18,71 @@ public static function configure(Schema $schema): Schema
         return $schema
             ->components([
                 Section::make('Menu settings')
-                    ->description('Only one active menu is used for each location. Saving an active menu deactivates other menus in the same location.')
+                    ->description('Only one active menu per location is used. Saving an active header, footer, mobile, or sidebar menu automatically deactivates other active menus in that location.')
                     ->schema([
                         TextInput::make('name')->required(),
                         Select::make('location')
-                            ->options(['header' => 'Header', 'footer' => 'Footer'])
+                            ->options([
+                                'header' => 'Header',
+                                'footer' => 'Footer',
+                                'mobile' => 'Mobile',
+                                'sidebar' => 'Sidebar',
+                            ])
+                            ->helperText('Header menu appears in the website header. Footer menu appears in the footer. Children appear as dropdown/submenu items.')
                             ->required(),
                         Toggle::make('is_active')->default(true),
-                        TextInput::make('sort_order')->numeric()->default(0),
+                        TextInput::make('sort_order')
+                            ->numeric()
+                            ->default(fn (): int => SortOrder::next(Menu::class))
+                            ->helperText('Used when choosing the active menu. New menus get the next order automatically.'),
                     ])
                     ->columns(2),
 
                 Section::make('Menu items')
-                    ->description('Use routes for system pages, page slugs for CMS pages, or full URLs for external links. Inactive items are hidden on the public site.')
+                    ->description('Create parent, child, and grandchild items. Use routes for system pages, page slugs for CMS pages, or URLs for external links. Inactive items are hidden on the public site.')
                     ->schema([
                         Repeater::make('items')
-                            ->schema([
-                                Toggle::make('is_active')->label('Visible')->default(true),
-                                TextInput::make('label.en')->label('English label')->required(),
-                                TextInput::make('label.fa')->label('Farsi label'),
-                                TextInput::make('label.ar')->label('Arabic label'),
-                                Select::make('type')
-                                    ->options(['route' => 'System route', 'page' => 'CMS page', 'url' => 'URL'])
-                                    ->default('route')
-                                    ->required(),
-                                TextInput::make('target')
-                                    ->required()
-                                    ->helperText('Examples: home, news.index, gallery.index, about, https://example.com.'),
-                            ])
+                            ->schema(self::itemSchema(1))
                             ->columns(2)
                             ->reorderable()
-                            ->addActionLabel('Add menu item')
+                            ->addActionLabel('Add top-level item')
                             ->columnSpanFull(),
                     ]),
             ]);
     }
+
+    /**
+     * @return array<int, mixed>
+     */
+    private static function itemSchema(int $level): array
+    {
+        $schema = [
+            Toggle::make('is_active')->label('Visible')->default(true),
+            TextInput::make('sort_order')
+                ->numeric()
+                ->helperText('Optional. Items are auto-ordered when left empty.'),
+            TextInput::make('label.en')->label('English label')->helperText('Leave empty only if you want the frontend fallback label.'),
+            TextInput::make('label.fa')->label('Farsi label'),
+            TextInput::make('label.ar')->label('Arabic label'),
+            Select::make('type')
+                ->options(['route' => 'System route', 'page' => 'CMS page', 'url' => 'URL'])
+                ->default('route')
+                ->required(),
+            TextInput::make('target')
+                ->helperText('Examples: home, news.index, gallery.index, about, google.com, https://example.com.')
+                ->required(),
+        ];
+
+        if ($level < 3) {
+            $schema[] = Repeater::make('children')
+                ->label($level === 1 ? 'Child items' : 'Grandchild items')
+                ->schema(self::itemSchema($level + 1))
+                ->columns(2)
+                ->reorderable()
+                ->addActionLabel($level === 1 ? 'Add child item' : 'Add grandchild item')
+                ->columnSpanFull();
+        }
+
+        return $schema;
+    }
 }

app/Filament/Resources/Menus/Tables/MenusTable.php

@@ -21,6 +21,7 @@ class MenusTable
     public static function configure(Table $table): Table
     {
         return TableUx::apply($table
+            ->defaultSort('sort_order')
             ->columns([
                 TextColumn::make('name')->searchable()->sortable()->wrap()->lineClamp(2),
                 TextColumn::make('location')->badge(),

app/Filament/Resources/NewsPosts/NewsPostResource.php

@@ -9,6 +9,7 @@
 use App\Filament\Resources\NewsPosts\Schemas\NewsPostForm;
 use App\Filament\Resources\NewsPosts\Schemas\NewsPostInfolist;
 use App\Filament\Resources\NewsPosts\Tables\NewsPostsTable;
+use App\Filament\Support\RequiresPermission;
 use App\Models\NewsPost;
 use BackedEnum;
 use Filament\Resources\Resource;
@@ -18,6 +19,8 @@
 
 class NewsPostResource extends Resource
 {
+    use RequiresPermission;
+
     protected static ?string $model = NewsPost::class;
 
     protected static string|BackedEnum|null $navigationIcon = Heroicon::OutlinedNewspaper;