call to reverseHeaders() function in jqm version 1.3 introduces a bug with tables #5589

Closed
urbien opened this Issue Feb 5, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@urbien

urbien commented Feb 5, 2013

I get $cells undefined. See repro here

I do not have cells attribute in my table.
See that this function in jqm assumes cells will be present.
The repro above is with 1.3.0 beta, but I checked code in rc1 and same bug is present there too.

reverseHeaders.each(function(i){
        var $cells = $( this ).jqmData( "cells" ),
            colstart = $( this ).jqmData( "colstart" ),
            hierarchyClass = $cells.not( this ).filter( "thead th" ).length && " ui-table-cell-
I added this code to stop in debugger but to continue to work without it:
   if ($cells = 'undefined') {
      debugger;                
      return;
   }
@toddparker

This comment has been minimized.

Show comment
Hide comment
@toddparker

toddparker Apr 2, 2013

Contributor

We will be adding refresh methods to both table plugins for 1.3.1 so closing this since it will be addressed by that change. See #5841 and #5842.

Contributor

toddparker commented Apr 2, 2013

We will be adding refresh methods to both table plugins for 1.3.1 so closing this since it will be addressed by that change. See #5841 and #5842.

@toddparker toddparker closed this Apr 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment