Merged pull request #1503 from eddiemonge/eddie_listview.

micro-optimizations and styling changes to listview.js
This commit is contained in:
Todd Parker 2011-04-27 13:34:39 -07:00
commit c2e65107c0

View file

@ -18,40 +18,32 @@ $.widget( "mobile.listview", $.mobile.widget, {
},
_create: function() {
var $list = this.element,
o = this.options;
var t = this;
// create listview markup
$list
.addClass( "ui-listview" );
if ( o.inset ) {
$list.addClass( "ui-listview-inset ui-corner-all ui-shadow" );
}
t.element.addClass(function( i, orig ) {
return orig + " ui-listview " + ( t.options.inset ? " ui-listview-inset ui-corner-all ui-shadow " : "" );
});
this.refresh();
t.refresh();
},
_itemApply: function( $list, item ) {
// TODO class has to be defined in markup
item.find( ".ui-li-count" )
.addClass( "ui-btn-up-" + ($list.jqmData( "counttheme" ) || this.options.countTheme) + " ui-btn-corner-all" );
item.find( "h1, h2, h3, h4, h5, h6" ).addClass( "ui-li-heading" );
item.find( "p, dl" ).addClass( "ui-li-desc" );
item.find("img:first-child:eq(0)").addClass( "ui-li-thumb" ).each(function() {
.addClass( "ui-btn-up-" + ($list.jqmData( "counttheme" ) || this.options.countTheme) + " ui-btn-corner-all" ).end()
.find( "h1, h2, h3, h4, h5, h6" ).addClass( "ui-li-heading" ).end()
.find( "p, dl" ).addClass( "ui-li-desc" ).end()
.find("img:first-child:eq(0)").addClass( "ui-li-thumb" ).each(function() {
item.addClass( $(this).is( ".ui-li-icon" ) ? "ui-li-has-icon" : "ui-li-has-thumb" );
});
item.find( ".ui-li-aside" ).each(function() {
}).end()
.find( ".ui-li-aside" ).each(function() {
var $this = $(this);
$this.prependTo( $this.parent() ); //shift aside to front for css float
});
},
_removeCorners: function(li){
_removeCorners: function( li ) {
li
.add( li.find(".ui-btn-inner, .ui-li-link-alt, .ui-li-thumb") )
.removeClass( "ui-corner-top ui-corner-bottom ui-corner-br ui-corner-bl ui-corner-tr ui-corner-tl" );
@ -73,8 +65,7 @@ $.widget( "mobile.listview", $.mobile.widget, {
$list.find( ".ui-li-dec" ).remove();
}
var numli = li.length;
for (var pos = 0; pos < numli; pos++) {
for (var pos = 0, numli = li.length; pos < numli; pos++) {
var item = li.eq(pos),
itemClass = "ui-li";
@ -83,9 +74,8 @@ $.widget( "mobile.listview", $.mobile.widget, {
continue;
}
var itemTheme = item.jqmData("theme") || o.theme;
var a = item.children( "a" );
var itemTheme = item.jqmData("theme") || o.theme,
a = item.children( "a" );
if ( a.length ) {
var icon = item.jqmData("icon");
@ -121,7 +111,7 @@ $.widget( "mobile.listview", $.mobile.widget, {
iconpos: false
})
.find( ".ui-btn-inner" )
.append( $( "<span>" ).buttonMarkup({
.append( $( "<span />" ).buttonMarkup({
shadow: true,
corners: true,
theme: splittheme,
@ -228,8 +218,8 @@ $.widget( "mobile.listview", $.mobile.widget, {
newPage.page();
var anchor = parent.find('a:first');
if (!anchor.length) {
anchor = $("<a></a>").html( nodeEls || title ).prependTo(parent.empty());
if ( !anchor.length ) {
anchor = $("<a />").html( nodeEls || title ).prependTo( parent.empty() );
}
anchor.attr('href','#' + id);
}).listview();