Skip to content

Commit d881169

Browse files
committed
Fix: Automatic disablement of FixedHeader based on visibility wouldn't reenable when needed
Was a result of aa203d6
1 parent 0f43b7d commit d881169

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

js/dataTables.fixedHeader.js

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,8 @@ var FixedHeader = function ( dt, config ) {
6060
header: -1,
6161
footer: -1
6262
},
63-
enable: true
63+
enable: true,
64+
autoDisable: false
6465
};
6566

6667
this.dom = {
@@ -143,10 +144,12 @@ $.extend( FixedHeader.prototype, {
143144
*
144145
* @param {boolean} enable `true` to enable, `false` to disable
145146
*/
146-
enable: function ( enable, update )
147+
enable: function ( enable, update, type )
147148
{
148149
this.s.enable = enable;
149150

151+
this.s.enableType = type;
152+
150153
if ( update || update === undefined ) {
151154
this._positions();
152155
this._scroll( true );
@@ -197,16 +200,19 @@ $.extend( FixedHeader.prototype, {
197200
*/
198201
update: function (force)
199202
{
200-
if (! this.s.enable) {
203+
var table = this.s.dt.table().node();
204+
205+
// Update should only do something if enabled by the dev.
206+
if (! this.s.enable && ! this.s.autoDisable) {
201207
return;
202208
}
203209

204-
var table = this.s.dt.table().node();
205-
206210
if ( $(table).is(':visible') ) {
211+
this.s.autoDisable = false;
207212
this.enable( true, false );
208213
}
209214
else {
215+
this.s.autoDisable = true;
210216
this.enable( false, false );
211217
}
212218

0 commit comments

Comments
 (0)