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.

Sign up for GitHub

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

Jump to bottom

Setting min-height for gridstack container makes the grid stack item not resizable #628

Closed
vbinithyanandamv opened this issue Apr 13, 2017 · 6 comments
Closed

Setting min-height for gridstack container makes the grid stack item not resizable #628

vbinithyanandamv opened this issue Apr 13, 2017 · 6 comments
Labels
type:bug

Comments

@vbinithyanandamv
Copy link

I have tried setting the min-height for my grid container. Once i set up a min-height, i can't able to shrink/resize the grid stack item properly. Can be reproduced simply in all demo examples by giving a grid-stack min-height.
@radiolips
Copy link
Member

@vbinithyanandamv I'm surprised this is the case. Could you please put together a jsfiddle to look at?

@vbinithyanandamv
Copy link
Author

Yes sure.Let me add a fiddle.

@vbinithyanandamv
Copy link
Author

vbinithyanandamv commented Apr 21, 2017 •
edited

Hi i have recreated the fiddle here

https://jsfiddle.net/divakarvenu/5ux1y3dr/2/

just try to resize.

@radiolips
Copy link
Member

Interesting! Thanks, I'll look into it.

dhagan added a commit to dhagan/kb that referenced this issue Jul 15, 2017
@dhagan
disable resize
96af9ff 
 /* setting min-height recommended by troolee
     gridstack/gridstack.js#230
     gridstack/gridstack.js#628 */
hover for next and cancel
change nav-bar style
@radiolips radiolips mentioned this issue Sep 9, 2017
Merged
@radiolips
Copy link
Member

Hey @vbinithyanandamv ! I believe I've fixed the issue with the latest develop commit. Could you pull in the latest code and test it? It was a very little thing but gave me hours of grief. Let me know if it's fixed now, please! And, please, if you find any more issues, make sure to log them!

Thanks,
dylan

@vbinithyanandamv
Copy link
Author

Cool sure :)

@adumesny adumesny mentioned this issue Sep 13, 2017
Merged
@yehhor yehhor mentioned this issue Oct 23, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type:bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@radiolips @vbinithyanandamv