Skip to content

Commit dd9ae48

Browse files
committed
Fix for issue jquery-archive#779
Added a ui-li-has-count class for listview items that contain a count bubble. Changed default padding-right to 25px adjusting it to 75px when count bubble is present.
1 parent eb1ff37 commit dd9ae48

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

js/jquery.mobile.listview.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,14 @@ $.widget( "mobile.listview", $.mobile.widget, {
3636
},
3737

3838
_itemApply: function( $list, item ) {
39+
var $countli = item.find( ".ui-li-count" );
40+
if ( $countli.length ) {
41+
item.addClass( "ui-li-has-count" );
42+
}
43+
$countli.addClass( "ui-btn-up-" + ( $list.jqmData( "counttheme" ) || this.options.countTheme ) + " ui-btn-corner-all" );
44+
3945
// TODO class has to be defined in markup
40-
item.find( ".ui-li-count" )
41-
.addClass( "ui-btn-up-" + ( $list.jqmData( "counttheme" ) || this.options.countTheme ) + " ui-btn-corner-all" ).end()
42-
.find( "h1, h2, h3, h4, h5, h6" ).addClass( "ui-li-heading" ).end()
46+
item.find( "h1, h2, h3, h4, h5, h6" ).addClass( "ui-li-heading" ).end()
4347
.find( "p, dl" ).addClass( "ui-li-desc" ).end()
4448
.find( ">img:eq(0), .ui-link-inherit>img:eq(0)" ).addClass( "ui-li-thumb" ).each(function() {
4549
item.addClass( $(this).is( ".ui-li-icon" ) ? "ui-li-has-icon" : "ui-li-has-thumb" );

themes/default/jquery.mobile.listview.css

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@ ol.ui-listview .ui-li-jsnumbering:before { content: "" !important; } /* to avoid
1616
.ui-listview-inset .ui-li { border-right-width: 1px; border-left-width: 1px; }
1717
.ui-li:last-child, .ui-li.ui-field-contain:last-child { border-bottom-width: 1px; }
1818
.ui-li>.ui-btn-inner { display: block; position: relative; padding: 0; }
19-
.ui-li .ui-btn-inner a.ui-link-inherit, .ui-li-static.ui-li { padding: .7em 75px .7em 15px; display: block; }
19+
.ui-li .ui-btn-inner a.ui-link-inherit, .ui-li-static.ui-li { padding: .7em 25px .7em 15px; display: block; }
2020
.ui-li-has-thumb .ui-btn-inner a.ui-link-inherit, .ui-li-static.ui-li-has-thumb { min-height: 60px; padding-left: 100px; }
2121
.ui-li-has-icon .ui-btn-inner a.ui-link-inherit, .ui-li-static.ui-li-has-icon { min-height: 20px; padding-left: 40px; }
22+
.ui-li-has-count .ui-btn-inner a.ui-link-inherit, .ui-li-static.ui-li-has-count { padding-right: 75px; }
2223
.ui-li-heading { font-size: 16px; font-weight: bold; display: block; margin: .6em 0; text-overflow: ellipsis; overflow: hidden; white-space: nowrap; }
2324
.ui-li-desc { font-size: 12px; font-weight: normal; display: block; margin: -.5em 0 .6em; text-overflow: ellipsis; overflow: hidden; white-space: nowrap; }
2425
.ui-li-thumb, .ui-li-icon { position: absolute; left: 1px; top: 0; max-height: 80px; max-width: 80px; }

0 commit comments

Comments
 (0)