这是我的代码。
但正如你所看到的,它的丑陋。
有人有更好的想法吗?
.delegate(".ui-icon-plus", "click", function() {
var d = $(this).parent().data("val");
var values = $(obj).val();
values.push(d);
$(obj).val(values);
$(this)
.removeClass('ui-icon-plus')
.addClass('ui-icon-minus')
.attr('title', 'Remove keyword from template')
.parent().appendTo('div .keyword-left .ui-widget');
})
.delegate(".ui-icon-minus", "click", function() {
var d = $(this).parent().data("val");
var values = $(obj).val();
var idx = values.indexOf(d); // Find the index
if(idx!=-1) values.splice(idx, 1); // Remove it if really found!
$(obj).val(values);
$(this)
.removeClass('ui-icon-minus')
.addClass('ui-icon-plus')
.attr('title', 'Use keyword in template')
.parent().appendTo('div .keyword-right .ui-widget');
});
答案 0 :(得分:1)
你有什么作品,你可以通过双重.toggleClass()
调用来减少它,但是有一个问题区域。在IE< 9中的数组中不提供.indexOf()
,因此我要在此处添加.indexOf()
或使用$.inArray()
,如下所示:
.delegate(".ui-icon-plus", "click", function() {
var d = $(this).parent().data("val");
var values = $(obj).val();
values.push(d);
$(obj).val(values);
$(this).toggleClass('ui-icon-plus ui-icon-minus')
.attr('title', 'Remove keyword from template')
.parent().appendTo('div .keyword-left .ui-widget');
})
.delegate(".ui-icon-minus", "click", function() {
var d = $(this).parent().data("val");
var values = $(obj).val();
var idx = $.inArray(d, values); // Find the index
if(idx!=-1) values.splice(idx, 1); // Remove it if really found!
$(obj).val(values);
$(this).toggleClass('ui-icon-plus ui-icon-minus')
.attr('title', 'Use keyword in template')
.parent().appendTo('div .keyword-right .ui-widget');
});