Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
stufarquhar opened this issue Jun 4, 2020 · 10 comments
Closed

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

stufarquhar opened this issue Jun 4, 2020 · 10 comments
Labels
enhancement New feature or request

Comments

@stufarquhar
Copy link

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

@domthomas-dev domthomas-dev added the enhancement New feature or request label Jun 4, 2020
@vbezruchkin
Copy link

Hi,

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

Thanks

@domthomas-dev
Copy link
Contributor

@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
Copy link

Hi,

Thanks for the fix. Worked fine for me.

image

Cheers

@domthomas-dev
Copy link
Contributor

@stufarquhar Have you got a feedback ?

@happyDemon
Copy link

happyDemon commented Jul 13, 2020

@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: eminiarts/nova-tabs#97
flex-wrap needed to be added to the fields container on their end

@domthomas-dev
Copy link
Contributor

@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
Copy link

happyDemon commented Jul 17, 2020

@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
Copy link
Contributor

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

@happyDemon
Copy link

happyDemon commented Jul 17, 2020

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

@domthomas-dev
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants