Skip to content

Commit f5a41be

Browse files
author
GrayYoung
committed
Fix default clear method, clear all inputs into none value.
1 parent 8b148b0 commit f5a41be

3 files changed

Lines changed: 13 additions & 17 deletions

File tree

dist/jquery.clearForm.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,14 @@
3030
};
3131

3232
if ($.inArray(element.type.toLowerCase(), _supporType.unitGroup) >= 0) {
33-
ui.item.prop('checked', ui.item.prop('defaultChecked'));
33+
ui.item.prop('checked', false);
3434
} else if ($.inArray(tagName, _supporType.textGroup) >= 0) {
35-
ui.item.val(ui.item.prop('defaultValue'));
35+
ui.item.val('');
3636
} else if ($.inArray(tagName, _supporType.listGroup) >= 0) {
37-
ui.item.children().each(function(index) {
38-
if(this.defaultSelected) {
39-
ui.item.prop('selectedIndex', index);
40-
return false;
41-
}
42-
});
37+
ui.item.val('');
38+
if(ui.item.prop('selectedIndex') === -1) {
39+
ui.item.prop('selectedIndex', 0);
40+
}
4341
}
4442
if ($.type(opts.after) === 'function') {
4543
opts.after(event, ui);

dist/jquery.clearForm.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/jquery.clearForm.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,14 @@
3030
};
3131

3232
if ($.inArray(element.type.toLowerCase(), _supporType.unitGroup) >= 0) {
33-
ui.item.prop('checked', ui.item.prop('defaultChecked'));
33+
ui.item.prop('checked', false);
3434
} else if ($.inArray(tagName, _supporType.textGroup) >= 0) {
35-
ui.item.val(ui.item.prop('defaultValue'));
35+
ui.item.val('');
3636
} else if ($.inArray(tagName, _supporType.listGroup) >= 0) {
37-
ui.item.children().each(function(index) {
38-
if(this.defaultSelected) {
39-
ui.item.prop('selectedIndex', index);
40-
return false;
41-
}
42-
});
37+
ui.item.val('');
38+
if(ui.item.prop('selectedIndex') === -1) {
39+
ui.item.prop('selectedIndex', 0);
40+
}
4341
}
4442
if ($.type(opts.after) === 'function') {
4543
opts.after(event, ui);

0 commit comments

Comments
 (0)