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

改后：
[img]http://hanwei59.iteye.com/upload/attachment/75404/9aad3a19-a1cd-311e-b80b-b99892f747a6.jpg[/img]
改之前：
[img]http://hanwei59.iteye.com/upload/attachment/75406/934e6d36-1068-303a-81df-af75337a53c2.jpg[/img]


安装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：[url]https://developer.mozilla.org/en/XUL_Tutorial/The_Box_Model[/url]
以及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次失败就不提了。 :lol: