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

Gap between panels leads to some panels' size overflow #613

Closed
artaommahe opened this issue May 24, 2024 · 5 comments · Fixed by #614 or #618
Closed

Gap between panels leads to some panels' size overflow #613

artaommahe opened this issue May 24, 2024 · 5 comments · Fixed by #614 or #618
Labels
bug Something isn't working

Comments

@artaommahe
Copy link

artaommahe commented May 24, 2024

Describe the bug
Having a layout with nested branches and gaps between panels (--dv-group-gap-size: 0.5rem;) leads to some panels' overflowing their expected boundaries (panels 4 and 5 in the screenshot)

image

To Reproduce

Steps to reproduce the behavior:

  1. Go to https://stackblitz.com/edit/vitejs-vite-wsic2s?file=src%2FApp.tsx,src%2Findex.css
  2. See that panels' 4 and 5 are bigger than expected and overflow over the gap between panels

image

Expected behavior

Nested panels respect gaps in their sizing calculation

Screenshots
^

Desktop (please complete the following information):

  • Browser Chrome
  • Version 124.0.6367.208

Additional context
-

@mathuo mathuo linked a pull request May 24, 2024 that will close this issue
@mathuo mathuo added the bug Something isn't working label May 26, 2024
@mathuo
Copy link
Owner

mathuo commented May 28, 2024

Released in 1.14.1

@artaommahe
Copy link
Author

artaommahe commented May 29, 2024

@mathuo the fix breaks group's custom styling (like border radius) 😅

you can check this at the demo page with the replit theme (i also made the gap bigger to showcase the issue)

after update
image

before (with border gap back to .view, border radius and color are back)
image

@mathuo mathuo linked a pull request May 29, 2024 that will close this issue
@mathuo
Copy link
Owner

mathuo commented May 29, 2024

@mathuo the fix breaks group's custom styling (like border radius) 😅

you can check this at the demo page with the replit theme (i also made the gap bigger to showcase the issue)

after update image

before (with border gap back to .view, border radius and color are back) image

damn, I guess this needs a bit more though. Let me have a think

@mathuo mathuo reopened this Jul 6, 2024
@mathuo
Copy link
Owner

mathuo commented Jul 11, 2024

1.15.0 now uses a prop gap defined in pixels . The CSS property --group-gap-size is no longer used.

Let me know if this does the trick.

@mathuo mathuo closed this as completed Jul 11, 2024
@artaommahe
Copy link
Author

artaommahe commented Jul 12, 2024

@mathuo looks like passing gap={6} to the DockviewReact component doesn't work - https://stackblitz.com/edit/vitejs-vite-9f1xvv?file=src%2FApp.tsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants