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
797 stars 104 forks source link

[3.x] modelResource Ломается верстка если обернуть метрики во фрагмент #1348

Closed ArtemYurov closed 2 days ago

ArtemYurov commented 3 days ago

MoonShine Version

3.x-dev 856a134

Laravel Version

11.34.2

PHP Version

8.3.12

Database Driver & Version

No response

Description

По мотивам https://t.me/moonshine_ru/6/13050

Верстка без использования фрагмента

image

Верстка с фрагментом

image

Вообще пожелание сделать по умолчанию фрагмент метрик с каким нибудь методом для получения его имени для события обновления

Steps To Reproduce

    protected function metrics(): array
    {
        return [
            Fragment::make([
                Column::make([
                    ValueMetric::make('Тест')
                        ->value(100)
                ])->columnSpan(3),
                Column::make([
                    ValueMetric::make('Тест')
                        ->value(100)
                ])->columnSpan(3),
            ])->name('metrics')
        ];
    }
lee-to commented 3 days ago

А где в примере Grid? без Grid же Column будут разваливаться

ArtemYurov commented 3 days ago

точно. но проблему это решает. вот с grid

image
lee-to commented 3 days ago

точно. но проблему это решает. вот с grid

image

Да там шаблон прям под метрики, чтобы не юзать колумны и гриды, получается сейчас нужно через компоненты страницы такое выводить а не через область метрик

lee-to commented 3 days ago

Или убрать готовые стили с флексами под метрики и дать возможность самим указывать гриды и колумны, но усложнит добавление метрик, сейчас этот блок для быстрого добавления только метрик без лишних компонентов. Вообщем надо подумать

ArtemYurov commented 3 days ago

ну я у себя добавил метрики с фрагментом в topLayer

но идея обновлять стоковые метрики по событию возникла не только у меня одного, поэтому issue

lee-to commented 2 days ago

https://github.com/moonshine-software/moonshine/pull/1354