codenco-dev / nova-grid-system

Nova grid system for Laravel Nova
80 stars 33 forks source link

Breaking layout when using eminiarts/nova-tabs package #5

Closed stufarquhar closed 4 years ago

stufarquhar commented 4 years ago

This simply does not work when using eminiarts/nova-tabs in detail view. It does however work without the use of tabs thou.

see image. Screen Shot 2020-06-04 at 1 13 45 PM

vbezruchkin commented 4 years ago

Hi,

I have the same problem. Do you expect any fix for this?

Thanks

domthomas-dev commented 4 years ago

@stufarquhar @vbezruchkin Can you check if this release fix your problem ?

https://github.com/codenco-dev/nova-grid-system/releases/tag/1.0.1

I don't know if the problem wasn't with breadcrumb for example.

Thanks for your feedback

vbezruchkin commented 4 years ago

Hi,

Thanks for the fix. Worked fine for me.

image

Cheers

domthomas-dev commented 4 years ago

@stufarquhar Have you got a feedback ?

happyDemon commented 4 years ago

@thomasdominic I still have an issue here, basically all fields are shown in 1 row when in tab-view, if I disable tabs there's no problem

I threw together a fix and sent a PR to nova-tabs, for the people interested: https://github.com/eminiarts/nova-tabs/pull/97 flex-wrap needed to be added to the fields container on their end

domthomas-dev commented 4 years ago

@happyDemon Can you post a new issue on this repo and give me an example of resource ? I will try to look at this

happyDemon commented 4 years ago

@thomasdominic the main difference with @vbezruchkin is that I used the TabsOnEdit trait on my resource (his screenshot just shows the standard nova form-layout).

Screenshot 2020-07-13 at 10 43 24

The issue is that nova-tabs uses a custom component based on PanelItem and the container element didn't have flex-wrap, which is why all fields were rendered on 1 line.

I sent a PR to nova-tabs, in the meantime I tagged my repo so I could use it in my own projects in the meantime, @eminiarts doesn't seem to be very responsive when I look at all the open PRs.

domthomas-dev commented 4 years ago

Okay :) I hope my package is cool for you :)

happyDemon commented 4 years ago

@thomasdominic this is a killer package :D it's so weird nova doesn't support this out of the box

domthomas-dev commented 4 years ago

@happyDemon hey thanks, don't hesitate to talk about it then :)