properly work with dynamically added AJAXed tabs

[kpect]

1) added "exists" method

exists: function(index) {
 if (typeof index == 'string')
  return this.$tabs.index( this.$tabs.filter('[href$=' + index + ']')[0] );
 if(this.$tabs.length <= index || 0 > index)
  return false;
 return index;
}

2) changed "tabId" because this.$tabs.filter() can't find hrefs with "." or ":" symbols.

tabId: function(a) {
 return a.title && a.title.replace(/\s/g, '_').replace(/[^A-Za-z0-9\-_]/g, '') || this.options.idPrefix + $.data(a);
},

3) "tabTemplate" option set to: '<li class="ui-tabs-nav-item"><a href="#{href}" title="#{href}"><span>#{label}</span></a></li>' for compatibilty with the UI Themes and constantly ids generation.

4) patch for "add" method

// try to find an existing element before creating a new one
if(-1 != this.exists('#'+id))
{
 this.select('#'+id);
 return;
}

5) patch fot "select" method

select: function(index) {
 index = this.exists(index);
 if(-1 != index)
  this.$tabs.eq(index).trigger(this.options.event);
},

[kpect]

I'm sorry, there is bug in exists method. Fix:

exists: function(index) {
 if (typeof index == 'string')
  return this.$tabs.index( this.$tabs.filter('[href$=' + index + ']')[0] );
 if(this.$tabs.length <= index || 0 > index)
  return -1;
 return index;
}

[klaus.hartl]

I'm sorry, I don't even know why this is necessary. Please reopen with a test case exposing the bug or at least a description of the erroneous behavior, e.g. what this is supposed fix.

[klaus.hartl]

"properly work with dynamically added AJAXed tabs" is not a sufficient description to me.

[Scott González]

Milestone TBD deleted