wrong counting of minHeight for resizable widget

[miroslav-peterka]

At line

gridstack.js/src/gridstack.js

Line 1281 in 17fcce8

var strictCellHeight = Math.ceil(o.outerHeight() / o.attr('data-gs-height'));

is some mistake, IMHO because you forgot to take in equation vertical margin.

example:
cellHeight = 80px and vertical margin = 20px
item height is 3, minHeight is 2

minHeight in pixels should be 80 x 2 + 20 = 180px. but at pointed line is counted number 188px

803 + 220 = 280
280 / 3 = 93.3 ~= 94
94 * 2 = 188px

[radiolips]

@miroslav-peterka you may certainly be correct. Could you provide an example that I could take a look at?
(edited for clarity)

[adumesny]

@miroslav-peterka you are absolutely right - here is the before when you can't drag smaller and after my fix (coming up) to take margin into account.

before

after

[adumesny]

fixed in upcoming 0.5.2 release