Skip to content

Conversation

@SebKay
Copy link
Owner

@SebKay SebKay commented Dec 8, 2025

Note

Switch Filament icons to Lucide, add the Lucide package, and update Dashboard/User resources to use new icon enums.

  • Filament UI:
    • Replace Heroicons with LucideIcon enums in app/Filament/Resources/Users/Tables/UsersTable.php (email status icons) and app/Filament/Resources/Users/UserResource.php (navigation icon).
    • Add custom app/Filament/Pages/Dashboard.php using LucideIcon::House and register it in app/Providers/Filament/AdminPanelProvider.php.
  • Dependencies:
    • Add codewithdennis/filament-lucide-icons to composer.json and update composer.lock (including technikermathe/blade-lucide-icons).
  • Docs/Rules:
    • Update .cursor/rules/laravel-boost.mdc to note Lucide as the default icon enum for Filament v4.

Written by Cursor Bugbot for commit 492a322. This will update automatically on new commits. Configure here.

@SebKay SebKay self-assigned this Dec 8, 2025
@SebKay
Copy link
Owner Author

SebKay commented Dec 8, 2025

@cursor review

@SebKay SebKay merged commit 660b46f into main Dec 8, 2025
1 check passed
@SebKay SebKay deleted the lucide-icons branch December 8, 2025 08:03
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Dashboard page registered twice causing potential duplicate navigation

The new App\Filament\Pages\Dashboard class is being registered twice - once via discoverPages() which auto-discovers all pages in app/Filament/Pages, and again explicitly via ->pages([Dashboard::class]). Previously, the explicit Dashboard::class import referred to Filament\Pages\Dashboard (the framework's base class), which was not subject to auto-discovery. After this change, both registrations point to the same custom class, which could result in duplicate navigation items or other unexpected behavior.

app/Providers/Filament/AdminPanelProvider.php#L36-L40

->discoverResources(in: app_path('Filament/Resources'), for: 'App\\Filament\\Resources')
->discoverPages(in: app_path('Filament/Pages'), for: 'App\\Filament\\Pages')
->pages([
Dashboard::class,
])

app/Filament/Pages/Dashboard.php#L1-L10

<?php
namespace App\Filament\Pages;
use CodeWithDennis\FilamentLucideIcons\Enums\LucideIcon;
class Dashboard extends \Filament\Pages\Dashboard
{
protected static string|\BackedEnum|null $navigationIcon = LucideIcon::House;
}

Fix in Cursor Fix in Web


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants