Skip to content

Commit 177be11

Browse files
Textinput: Only set more padding right if option clear button is true. Fixes jquery-archive#5470.
1 parent 41cffcb commit 177be11

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

css/structure/jquery.mobile.forms.textinput.css

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@ div.ui-input-text input.ui-input-text, div.ui-input-text textarea.ui-input-text,
55
.ui-input-search input.ui-input-text { border: none; width: 100%; padding: .4em 0; margin: 0; display: block; background: transparent none; outline: 0 !important; }
66
.ui-input-search, div.ui-input-text { margin: .5em 0; background-image: none; position: relative; }
77
.ui-input-search { padding: 0 30px; }
8-
div.ui-input-text { padding: 0 30px 0 .4em; }
8+
div.ui-input-text { padding: 0 .4em; }
9+
div.ui-input-has-clear { padding: 0 30px 0 .4em; }
910
input.ui-input-text.ui-mini, textarea.ui-input-text.ui-mini,
1011
.ui-input-search.ui-mini, div.ui-input-text.ui-mini { margin: .25em 0; }
1112
.ui-field-contain input.ui-input-text, .ui-field-contain textarea.ui-input-text,

js/widgets/forms/textinput.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ $.widget( "mobile.textinput", $.mobile.widget, {
4646

4747
$( "label[for='" + input.attr( "id" ) + "']" ).addClass( "ui-input-text" );
4848

49-
focusedEl = input.addClass("ui-input-text ui-body-"+ theme );
49+
focusedEl = input.addClass( "ui-input-text ui-body-"+ theme );
5050

5151
// XXX: Temporary workaround for issue 785 (Apple bug 8910589).
5252
// Turn off autocorrect and autocomplete on non-iOS 5 devices
@@ -87,6 +87,10 @@ $.widget( "mobile.textinput", $.mobile.widget, {
8787
shadow: true,
8888
mini: o.mini
8989
});
90+
91+
if ( !isSearch ) {
92+
focusedEl.addClass( "ui-input-has-clear" );
93+
}
9094

9195
toggleClear();
9296

0 commit comments

Comments
 (0)