openpsa / jsgrid

Fork of last jqGrid version before license change
http://openpsa.github.io/jsgrid/
Other
28 stars 12 forks source link

Subgrid collapse doesn't work #87

Closed flack closed 9 years ago

flack commented 9 years ago

See

http://openpsa.github.io/grid.js/demos/kitchensink.html

you can load the subgrid, but it isn't removed when you close it. So when you open it again, it is displayed twice

bouks commented 9 years ago

Is it the same if there is datas in subgrids ?

flack commented 9 years ago

Haven't tried. But it works in Oleg's demo (on which I've based mine): http://www.ok-soft-gmbh.com/jqGrid/OK/grouping1.htm

flack commented 9 years ago

oops, just saw that this comes from here: https://github.com/openpsa/jsgrid/commit/4a1a83e14545bf78cc9ba3094e3846c4cf508e78. So my change has probably broken the fix. Version 2 will be forthcoming..

flack commented 9 years ago

Ok, should be fixed now. The problem was simply that the row ids in this particular example looked like 12.0, this was then used to construct a jQuery selector looking like #grid_12.0_expandedContent, which of course is interpreted as "element with id grid_12 and class 0_expandedContent. So the string simply needed escaping