From 1ffd4a0a207ecc6ad3bd23354ee04ce7b2fe7199 Mon Sep 17 00:00:00 2001 From: Mottie Date: Wed, 4 Feb 2015 07:29:07 -0600 Subject: [PATCH] Pager: update goto independently of other elements. Fixes #811 --- addons/pager/jquery.tablesorter.pager.js | 16 ++++++++-------- js/widgets/widget-pager.js | 17 ++++++++--------- 2 files changed, 16 insertions(+), 17 deletions(-) diff --git a/addons/pager/jquery.tablesorter.pager.js b/addons/pager/jquery.tablesorter.pager.js index e509578b..d9406672 100644 --- a/addons/pager/jquery.tablesorter.pager.js +++ b/addons/pager/jquery.tablesorter.pager.js @@ -193,16 +193,16 @@ } return extra.length > 1 && data && data[extra[0]] ? data[extra[0]][extra[1]] : p[str] || (data ? data[str] : deflt) || deflt; }); + if ( p.$goto.length ) { + t = ''; + $.each(buildPageSelect(p), function(i, opt){ + t += ''; + }); + // innerHTML doesn't work in IE9 - http://support2.microsoft.com/kb/276228 + p.$goto.html(t).val( p.page + 1 ); + } if ($out.length) { $out[ ($out[0].tagName === 'INPUT') ? 'val' : 'html' ](s); - if ( p.$goto.length ) { - t = ''; - $.each(buildPageSelect(p), function(i, opt){ - t += ''; - }); - // innerHTML doesn't work in IE9 - http://support2.microsoft.com/kb/276228 - p.$goto.html(t).val( p.page + 1 ); - } // rebind startRow/page inputs $out.find('.ts-startRow, .ts-page').unbind('change').bind('change', function(){ var v = $(this).val(), diff --git a/js/widgets/widget-pager.js b/js/widgets/widget-pager.js index 966c10ba..8d97534d 100644 --- a/js/widgets/widget-pager.js +++ b/js/widgets/widget-pager.js @@ -430,17 +430,16 @@ tsp = ts.pager = { } return extra.length > 1 && data && data[extra[0]] ? data[extra[0]][extra[1]] : p[str] || (data ? data[str] : deflt) || deflt; }); - + if ( p.$goto.length ) { + t = ''; + $.each(tsp.buildPageSelect(p, c), function(i, opt){ + t += ''; + }); + // innerHTML doesn't work in IE9 - http://support2.microsoft.com/kb/276228 + p.$goto.html(t).val( p.page + 1 ); + } if ($out.length) { $out[ ($out[0].tagName === 'INPUT') ? 'val' : 'html' ](s); - if ( p.$goto.length ) { - t = ''; - $.each(tsp.buildPageSelect(p, c), function(i, opt){ - t += ''; - }); - // innerHTML doesn't work in IE9 - http://support2.microsoft.com/kb/276228 - p.$goto.html(t).val( p.page + 1 ); - } // rebind startRow/page inputs $out.find('.ts-startRow, .ts-page').off('change').on('change', function(){ var v = $(this).val(),