Hakimuddin-Sabunwala / jquery-datatables-row-grouping

Automatically exported from code.google.com/p/jquery-datatables-row-grouping
0 stars 3 forks source link

Combining Expandable grouping and 2 levels of grouping doesn't work (enhancement attached) #32

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set both bExpandableGrouping and bExpandableGrouping2 to true
2. Collapse a main group
3. Expand main group
4. Click on sub-group heading

What is the expected output? What do you see instead?
Expected:
1) When collapsing a main group heading, expect all rows down to the next main 
group to be hidden. 
2) When clicking on sub-group heading, rows below heading down to next 
sub-group heading are hidden

Actual:
1) sub-group headings remain visible
2) no collapse expand on sub-groups

What version of the product are you using? On what operating system?

1.1.6 on Windows XP

Please provide any additional information below

Thank you for this plugin - I found it not only very useful, but well written 
and easy to modify to achieve what I needed.  I have attached my updated 
version where 2 levels of grouping work as I expect them to.  Please feel free 
to use or discard as you see fit (note that I have not tested this in a paged 
table, nor with filtering  - my use case was for a single page of data only).  
Zip file contains modified source and change log. 

Original issue reported on code.google.com by michelle...@gmail.com on 9 Jul 2012 at 11:42

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by joc...@gmail.com on 13 Jul 2012 at 10:52

GoogleCodeExporter commented 9 years ago
Hi Michelle,

Thank you very much for this enhancement. I have included it in the version 
1.2.2. However if you look at the code you will see that it is refactored so it 
was not just a copy. First functions for creating, collapsing, and expanding 
groups are moved into the separate functions therefore code is divided.
Also, I have changed a logic for expanding. Currently there is a array of 
objects aoGroups that contain information about each group including the 
state(collapsed/expanded). If you make any additional changes in the new 
versions please use this structure.

You can see it on the 
http://jquery-datatables-row-grouping.googlecode.com/svn/trunk/twoLevelExpandabl
e.html.

Thank you again for this enhancement.

Jovan

Original comment by joc...@gmail.com on 15 Jul 2012 at 11:36