From 5b3850140802faf91af05cab52bca71a3101bf41 Mon Sep 17 00:00:00 2001
From: Mottie <wowmotty@gmail.com>
Date: Tue, 25 Feb 2014 09:34:03 -0600
Subject: [PATCH] Grouping widget collapsed option shows headers. Fixes #514

---
 docs/example-widget-grouping.html |  2 +-
 js/widgets/widget-grouping.js     | 25 +++++++++++++------------
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/docs/example-widget-grouping.html b/docs/example-widget-grouping.html
index 4c30478d..11fcc149 100644
--- a/docs/example-widget-grouping.html
+++ b/docs/example-widget-grouping.html
@@ -108,7 +108,7 @@ tr.group-header.collapsed td i {
 				if (column === 2) {
 					var subtotal = 0;
 					$rows.each(function(){
-						subtotal += parseInt( $(this).find("td").eq(column).text() );
+						subtotal += parseFloat( $(this).find("td").eq(column).text() );
 					});
 					$cell.find(".group-count").append("; subtotal: " + subtotal );
 				}
diff --git a/js/widgets/widget-grouping.js b/js/widgets/widget-grouping.js
index 76f9bbdd..f829624f 100644
--- a/js/widgets/widget-grouping.js
+++ b/js/widgets/widget-grouping.js
@@ -70,9 +70,6 @@ ts.grouping = {
 				cache = c.cache[tbodyIndex].normalized;
 				group = ''; // clear grouping across tbodies
 				$rows = c.$tbodies.eq(tbodyIndex).children('tr').not('.' + c.cssChildRow);
-				if (wo.group_collapsed && wo.group_collapsible) {
-					$rows.addClass('group-hidden');
-				}
 				for (rowIndex = 0; rowIndex < $rows.length; rowIndex++) {
 					if ( $rows.eq(rowIndex).is(':visible') ) {
 						// group class finds "group-{word/separator/letter/number/date/false}-{optional:#/year/month/day/week/time}"
@@ -103,14 +100,15 @@ ts.grouping = {
 					}
 				}
 			}
-			$rows = c.$table.find('tr.group-header').bind('selectstart', false);
-			if (wo.group_count || $.isFunction(wo.group_callback)) {
-				$rows.each(function(){
-					var $rows,
-						$row = $(this),
-						$label = $row.find('.group-count');
+			c.$table.find('tr.group-header')
+			.bind('selectstart', false)
+			.each(function(){
+				var $label,
+					$row = $(this),
+					$rows = $row.nextUntil('tr.group-header').filter(':visible');
+				if (wo.group_count || $.isFunction(wo.group_callback)) {
+					$label = $row.find('.group-count');
 					if ($label.length) {
-						$rows = $row.nextUntil('tr.group-header').filter(':visible');
 						if (wo.group_count) {
 							$label.html( wo.group_count.replace(/\{num\}/g, $rows.length) );
 						}
@@ -118,8 +116,11 @@ ts.grouping = {
 							wo.group_callback($row.find('td'), $rows, column, table);
 						}
 					}
-				});
-			}
+				}
+				if (wo.group_collapsed && wo.group_collapsible) {
+					$rows.addClass('group-hidden');
+				}
+			});
 			c.$table.trigger(wo.group_complete);
 			if (c.debug) {
 				$.tablesorter.benchmark("Applying groups widget: ", time);