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

Bug in Tabs #1169

Closed MadZee-M closed 2 months ago

MadZee-M commented 2 months ago

MoonShine Version

2

Laravel Version

10

PHP Version

8.3

Database Driver & Version

No response

Description

Здравствуйте! Нашел баг с Tabs. Если есть вложенные Tabs с ->vertical(), то дочерние Tabs не отображаются "Вертикально"

Steps To Reproduce

Tabs::make([
  Tab::make('Таб 1')->fields([
      Text::make('Имя')
  ]),
  Tab::make('Таб 2')->fields([
      Tabs::make([
          Tab::make('Sub tаб 1')->fields([
              Text::make('Имя')
          ]),
          Tab::make('Sub tаб 2')->fields([
              Text::make('Имя')
          ]),
      ])->vertical()
  ]),
])->vertical(),

Когда переключаю на Tab 2, то получаю горизонтальные Вкладки (вместо вертикальных)

image

А если изменить размер окна браузера, то получаю ожидаемый результат.

image

DissNik commented 2 months ago

может быть просто не умещаются, по умолчанию ширина блока 480px, если меньше то вертикальные вкладки отображаться не будут, можно попробовать задать ширину

Tabs::make([
    //...
])
    ->customAttributes([ 'data-tabs-vertical-min-width' => 300]) 
MadZee-M commented 2 months ago

может быть просто не умещаются, по умолчанию ширина блока 480px, если меньше то вертикальные вкладки отображаться не будут, можно попробовать задать ширину

Tabs::make([
    //...
])
    ->customAttributes([ 'data-tabs-vertical-min-width' => 300]) 

А почему, если я дочерние "Вкладки", перенесу наверх (сделаю родителями), то срабатывает как положено?

Даже на скриншотах видно, что родительские Вкладки имеют меньше ширины, чем дочерние.

Даже если так, то почему срабатывают при изменения размера окна браузера (как я понял .trigger('resize'))

DissNik commented 2 months ago

дочерние вкладки не могут быть больше чем родительский блок

DissNik commented 2 months ago

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

MadZee-M commented 2 months ago

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

Верно