修复jquery.treeview的增加子节点的方法的bug

  1.修复理由

  在一个android项目中用到了treeview控件(本来自己通过android的原生api实现了一个http://www.cnblogs.com/Mr-Nobody/p/3527688.html,但是动画效果总是有点小问题,实在是击碎我那小小的自信心,于是果断转用webview+js插件实现了。)。本来js插件里优秀的tree控件很多,比如国内著名的ztree,还有各大ui控件的自带tree。不过要数最简单易用就要数最为经典的jquery.treeview了。而且动画效果也还行,我就是看中了其轻便,动画效果好,而且也比较容易用。。(其它的tree暂时还没有接触过。。)。,所以就采用了它的。

  2.修复过程

在jquery.treeview中增加子节点的方法是:

    var branch=$("<li><span>child1</span></li>").appendTo(parentLi);
        $("#tree").treeview({
            add:branch
        });
View Code

但是此方法有个很严重的问题,就是增加的节点必须只能是li。如果是在一个没有子节点的节点中插入子节点时,该节点的展开按钮是没有的,也就是父级的事件没有绑定上去。。

于是乎照着这个思路写了几个hack的写法,都没有用,于是一狠心就上了源代码。。还好大神的代码不多,虽然以自己目前的水平看懂大神的代码还有几个地球到月球的距离,但是

好在大神代码不多,用ctrl+F也搜索到了。

略加猜测,便在这里加上了几行代码。。

 

 

居然人品爆发,一次成功了!

 

 

  3.补遗

  也许菜鸟的人品毕竟有限,在运气好的情况下对于增加节点的bug一次就修复了,但是在删除节点时,由于本人对jquery的事件机制一知半解,结果浪费了很多时间。。直到查看文档才恍然大悟。。

 

 官网的demo都会有"全部删除"的bug,真是对开源项目太不认真了。。

原因是jquery的冒泡事件导致的。

return  proxied.apply(this, arguments).bind("add", function(event, branches) {
            $(branches).prev()
                .removeClass(CLASSES.last)
                .removeClass(CLASSES.lastCollapsable)
                .removeClass(CLASSES.lastExpandable)
            .find(">.hitarea")
                .removeClass(CLASSES.lastCollapsableHitarea)
                .removeClass(CLASSES.lastExpandableHitarea);
            $(branches).find("li").andSelf().prepareBranches(settings).applyClasses(settings, $(this).data("toggler"));
                /*修复增加ul的bug*/
            if(branches.is("ul")){
                $(branches).parent().prepareBranches(settings).applyClasses(settings, $(this).data("toggler"));
            }
            return false;
        }).bind("remove", function(event, branches) {
            var prev = $(branches).prev();
            var parent = $(branches).parent();
            $(branches).remove();
            prev.filter(":last-child").addClass(CLASSES.last)
                .filter("." + CLASSES.expandable).replaceClass(CLASSES.last, CLASSES.lastExpandable).end()
                .find(">.hitarea").replaceClass(CLASSES.expandableHitarea, CLASSES.lastExpandableHitarea).end()
                .filter("." + CLASSES.collapsable).replaceClass(CLASSES.last, CLASSES.lastCollapsable).end()
                .find(">.hitarea").replaceClass(CLASSES.collapsableHitarea, CLASSES.lastCollapsableHitarea);
                /*修复全部删除的bug*/
                if(parent.children().length==0){
                    
                    parent.parent().removeClass(CLASSES.collapsable).removeClass(CLASSES.expandable)
                    parent.siblings(".hitarea").andSelf().remove();
                }

//返回false,终止冒泡,就不会把整棵树都删除了。。
                return false;
                //throw 1;
        });
View Code

 

 

posted @ 2014-01-22 23:39  Zane Young  阅读(1703)  评论(0编辑  收藏  举报