使用单击函数jQuery追加HTML无法正常工作

时间:2013-02-18 15:23:38

标签: javascript jquery

http://jsfiddle.net/CCKUz/

所以我有一个<span class="open">的框,它会增加可折叠div的高度以查看内容。它只能通过CSS高度折叠,而开放式点击功能可将高度设置为自动。容易,这是有效的。

当我附加开放和关闭跨度时,问题就出现了。当我将它们包含在实际的html中时,它们可以正常工作。当我追加它们时,它们不再起作用。我想也许这可能是由于js无法将.click函数应用于它们,因为它们是在加载后创建的,但即使创建它们并在同一函数中应用.click也没有帮助解决这个问题。

你有什么可能影响这个吗?感谢。

HTML:

<div class="box collapsible">
    <h3>Title</h3>
    <p>This is a sample paragraph that is here for placer purposes.</p>
</div>

CSS:

.box { height: 20px; border: 1px solid #000000; padding: 10px; margin: 20px; position:   relative; overflow: hidden; }
h3 { margin: 0 0 20px 0; line-height: 20px; }
.open, .close { text-indent: -9999px; width: 20px; height: 20px; position: absolute; top:  10px; right: 10px; }
.open { background: #00ff00; }
.close { background: #0000ff; }

JS:

$(function(){
        var box = $(".collapsible");
        var close = $(".collapsible span.close");
        var open = $(".collapsible span.open");
        box.each(function(){
            box.append('<span class="open">Open</span>');
            open.each(function(i){
                open.click(function(){
                    alert("You clicked open");
                    $(this).parent(box).css("height","auto").append('<span class="close">Close</span>');
                    $(this).hide();
                    $(this).parent().find(close).show();
                });
            });
            close.each(function(i){
                close.click(function(){
                    $(this).parent(box).css("height","15px");
                    $(this).hide();
                    $(this).parent().find(open).show();
                });
            });
        });
    });

4 个答案:

答案 0 :(得分:2)

不需要循环,jQuery在内部执行,您需要使用动态元素委托事件处理程序,如下所示:

$(function () {
    var box = $(".collapsible");
    box.append( $('<span />', {'class':'open'}) )
       .on('click', '.close', function () {
         $(this).hide()
                .closest('.collapsible')
                .css("height", "auto")
                .append( $('<span />', {'class':'close'}).show() );
    })
      .on('click', '.close', function () {
          $(this).hide()
                 .closest('.collapsible')
                 .css("height", "15px")
                 .find('.open').show();
    });
});

答案 1 :(得分:1)

在添加前选择open元素:

var open = $(".collapsible span.open");

试试这个:

$(function(){
        var box = $(".collapsible");
        var close = $(".collapsible span.close");

        box.each(function(){
            box.append('<span class="open">Open</span>');
            var open = $(".collapsible span.open"); //do it here!!
            open.each(function(i){
                open.click(function(){
                    alert("You clicked open");
                    $(this).parent(box).css("height","auto").append('<span class="close">Close</span>');
                    $(this).hide();
                    $(this).parent().find(close).show();
                });
            });
            close.each(function(i){
                close.click(function(){
                    $(this).parent(box).css("height","15px");
                    $(this).hide();
                    $(this).parent().find(open).show();
                });
            });
        });
    });

答案 2 :(得分:0)

.each无需委托功能

$(document).on('click','.collapsible span.open',function(){

演示http://jsfiddle.net/CCKUz/2/

答案 3 :(得分:0)

问题是您是动态添加.close跨度,因此您的初始搜索将找不到任何内容,因此不会向其添加任何点击处理程序。您可以使用事件委派轻松解决此问题。

.on('click', 'span.close', function(evt){

您还可以大量简化代码:

$(".collapsible")
  .append('<span class="open">Open</span>')
  .on('click', 'span.open', function(evt){
    // Search within the parent '.collapsible'.
    $(evt.delegateTarget)
      .css("height", "auto")
      .append('<span class="close">Close</span>');

    // Hide the '.open' span.
    $(this).hide();
  })
  .on('click', 'span.close', function(evt){
    // Search within the parent '.collapsible'.
    $(evt.delegateTarget)
      .css("height","15px");
      .find('.open').show();

    // Remove the old '.close' button since the
    // open handler will make a new one.
    $(this).remove();    
  })