ducksboard / gridster.js

gridster.js is a jQuery plugin that makes building intuitive draggable layouts from elements spanning multiple columns
http://gridster.net/
MIT License
6.03k stars 1.2k forks source link

Issue when building from serialzed grid layout #359

Closed FamiComplex closed 10 years ago

FamiComplex commented 10 years ago

In my testing, I found a small issue - when building a grid from existing serialized widgets, once the grid reaches a certain height element jump to the top of the page.

The issue is in the add_widget function, specifically, line 250, "this.add_faux_rows(pos.size_y);". pos has no "size_y" parameter when column and row values exist.

Changing this line to "this.add_faux_rows(size_y)" or readding those parameters to pos fixes the issue. Thanks!

vieron commented 10 years ago

Thanks! Fixed in #342 and will be merged soon