moonshine-software / moonshine

Laravel Admin panel and more. Simple for beginners and powerful for experts. Using Blade, Alpine.js and Tailwind CSS.
https://moonshine-laravel.com
MIT License
759 stars 100 forks source link

feat: optional avatar #1176

Closed finagin closed 2 months ago

finagin commented 2 months ago

Не во всех проектах есть аватарки, при этом заглушка съедает значительное пространство, поэтому предлагаю при явном указании false скрывать этот блок.

Profile::make(
    avatar: false,
    username: fn () => auth()->id() . ' | ' . auth()->user()?->email
)
Снимок экрана 2024-08-11 в 11 56 42

Плюс добавлена возможность передать Closure как в #1174

lee-to commented 2 months ago

Здесь теперь более явно присутствует не приятный момент требующий рефакторинга (в целом он давно напрашивался), заключается в том что если мы переопределяем avatar то не смотря до этого все равно делаем операции с полем из юзера в соответствии с конфигом Нужно если аватар переопределен (не null) тогда мы идем по пути работы с переопределенным значением

lee-to commented 2 months ago

Да и давай в замыкание передавать текущий контекст

lee-to commented 2 months ago

По этим двум пр можно подсмотреть реализацию в 3.x ветке