DataTables / Responsive

Responsive extension for DataTables, providing support for complex tables on all device screen sizes
Other
148 stars 86 forks source link

loadingRecords is not working with responsive plugin. #102

Closed parth67 closed 6 years ago

parth67 commented 6 years ago

loadingRecords is not working with responsive plugin.

Details: Inside _constructor _resize is called before dt is initialize.

// First pass - draw the table for the current viewport size
this._resize();

which internally calls dt.draw() and increments iDraw counter.

Condition to use loadingRecords is like

if ( oSettings.iDraw == 1 &&  _fnDataSource( oSettings ) == 'ajax' )
{
     sZero = oLang.sLoadingRecords;
}
DataTables commented 6 years ago

Could you confirm what version of Responsive you are using please? It should actually work okay with the code in git at the moment, although there is a bug in the current release that would cause it not to work.

parth67 commented 6 years ago

Responsive Plugin Version: 2.2.0 DT version: 1.10.16

DataTables commented 6 years ago

Could you try it with the code from the head of the master branch please? I think this should already have been resolved.

parth67 commented 6 years ago

I think commit in 55fc24b will resolve issue. What is timeline for this to be available in npm registry as new release

DataTables commented 6 years ago

Probably early next month.

parth67 commented 6 years ago

Thanks.