改了“JavaEye的Firefox插件”的一个bug

改后：

改之前：



安装Javaeye的Firefox插件0.3.3版时，发现了一个bug：标签过多时，对话框超出了屏幕的范围。反正在这没有情人的情人节也无聊，就研究一下看能改好不。

把插件的后缀由.xpi改成.zip解压，chrome目录下的javaeye_plugin.jar再解压，
进入content，找了一下发现：“添加收藏”那个对话框的配置文件就是bookmark-editor.xul，放置“标签”的地方就是这里：

                <vbox hidden="true">
                    <hbox flex="1">
                        <label value="已有的标签" />
                        <label value="点击挑选" class="no-left-margin" disabled="true" />
                    </hbox>
                    <hbox flex="1" id="saved_tags">
                        
                    </hbox>
                </vbox>

关于Box：https://developer.mozilla.org/en/XUL_Tutorial/The_Box_Model
以及bookmark-manager.js里的

    _createTagLabel: function(tag) {
        var lbl = document.createElement("label");
        lbl.setAttribute("value", tag[0] + " (" + tag[1] + ")" );
        lbl.setAttribute("onclick", "jeBookmarkManager.tagClick('" + tag[0] +"');");
        lbl.setAttribute("class", "label-link");
        lbl.setAttribute("tabindex", "-1" );
        $('saved_tags').appendChild(lbl);
    },

调用时：

            for (i = 0; i < result.length; i++) {
                this._createTagLabel(result[i]);
            }

hbox里的元素是水平放置的，vbox里的元素是垂直放置的。难怪当我的标签太多时屏幕就显示不下了。
改成vbox里嵌套hbox就应该OK了。于是bookmark-editor.xul里修改如下：

                <vbox hidden="true">
                    <hbox flex="1">
                        <label value="已有的标签" />
                        <label value="点击挑选" class="no-left-margin" disabled="true" />
                    </hbox>
		    <vbox id="saved_tags">

		    </vbox>
                </vbox>

再改bookmark-manager.js：

    _createTagLabel: function(hbox,tag) {
        var lbl = document.createElement("label");
        lbl.setAttribute("value", tag[0] + " (" + tag[1] + ")" );
        lbl.setAttribute("onclick", "jeBookmarkManager.tagClick('" + tag[0] +"');");
        lbl.setAttribute("class", "label-link");
        lbl.setAttribute("tabindex", "-1" );
        hbox.appendChild(lbl);
    },
	_createTags: function(result) {
		var length = result.length;
		var SIZE = 10;
		var rows = 0;
		var i = 0;
		while(i<length){
			var hbox = document.createElement("hbox");
			for(;i<(rows+1)*SIZE;i++){
				if(i>=length) break;
				this._createTagLabel(hbox,result[i]);
			}
			$('saved_tags').appendChild(hbox);
			rows += 1;
			i = rows*SIZE;
		}
    },

调用的地方：

this._createTags(result);

OK了，效果如文章开头所示，当然其中经历的n次失败就不提了。

评论

8 楼 远去的渡口 2010-01-27  

真佩服呀！偶尔也发现过一些比较知名网站的bug，但是还没有想过把它改好。。。。

7 楼 QuakeWang 2009-02-20  

新的版本已经解决这个问题，再次感谢

6 楼 hanwei59 2009-02-15  

robbin 写道

QuakeWang 写道十分感谢，下次升级插件的时候会将你的这个更新放进去。建议将楼主作为contributor之一写入插件说明文档。

这。。。不用了吧

5 楼 robbin 2009-02-15  

QuakeWang 写道

十分感谢，下次升级插件的时候会将你的这个更新放进去。

建议将楼主作为contributor之一写入插件说明文档。

4 楼 hanwei59 2009-02-15  

ibio 写道

很强悍，顶一个！~LZ辛苦了。呵呵！~

Thanks

3 楼 ibio 2009-02-14  

很强悍，顶一个！~LZ辛苦了。呵呵！~

2 楼 hanwei59 2009-02-14  

QuakeWang 写道

十分感谢，下次升级插件的时候会将你的这个更新放进去。

呵呵 很荣幸

1 楼 QuakeWang 2009-02-14  

十分感谢，下次升级插件的时候会将你的这个更新放进去。