<首选项窗口>当 browser.preferences.animateFadeIn == true 时，将自身调整到错误的选项卡

Question

我的扩展程序的首选项屏幕遇到了（又一个）错误。当 browser.preferences.animateFadeIn 设置为 true 时（就像在 Mac 上一样），窗口大小应该更改以完全适合切换窗格上的内容。相反，当窗口打开时，它的大小会调整为最大窗格，但在切换窗格时它会发生尽可能多的变化。如果还不太清楚，这里有一个截屏视频：http://files.droplr。 com/files/22337488/4IVT.ScreenFlow.mov

即使删除所有

<?xml version="1.0"?>
<?xml-stylesheet href="chrome://global/skin/" type="text/css"?>
<?xml-stylesheet href="chrome://browser/skin/preferences/preferences.css" type="text/css"?>
<?xml-stylesheet href="chrome://nextplease/skin/nextpleasePreferences.css" type="text/css"?>

<!DOCTYPE window SYSTEM "chrome://nextplease/locale/nextplease.dtd">
<prefwindow id="nextpleaseprefs" title="&options.title;"
    xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">

    <prefpane id="nextplease.images" label="&options.images.title;" image="chrome://nextplease/skin/Image.png">
        <preferences>
            <preference id="nextimages" name="nextplease.nextimage.expr0" type="unichar"/>
            <preference id="previmages" name="nextplease.previmage.expr0" type="unichar"/>
            <preference id="firstimages" name="nextplease.firstimage.expr0"  type="unichar"/>
            <preference id="lastimages" name="nextplease.lastimage.expr0" type="unichar"/>
        </preferences>
        <hbox flex="1">
            <listbox width="80" onselect="nextplease.selectedPanelChanged(this);">
                <listitem label="&options.next;" selected="true"/>
                <listitem label="&options.prev;"/>
                <listitem label="&options.first;"/>
                <listitem label="&options.last;"/>
            </listbox>
            <separator class="groove" orient="vertical" style="opacity: 0.5; margin-top: 5px; margin-bottom: 5px;"/>
            <vbox flex="1">
                <deck flex="1">
                    <listbox id="Next_Image_list" seltype="multiple" flex="1"
                        onkeypress="nextplease.removeSelectedOnDelete(event, this);" onselect="nextplease.enableDisableRemoveButton(this);" onchange="nextplease.syncListboxToPref(this);"/>
                    <listbox id="Prev_Image_list" seltype="multiple" flex="1"
                        onkeypress="nextplease.removeSelectedOnDelete(event, this);" onselect="nextplease.enableDisableRemoveButton(this);" onchange="nextplease.syncListboxToPref(this);"/>
                    <listbox id="First_Image_list" seltype="multiple" flex="1"
                        onkeypress="nextplease.removeSelectedOnDelete(event, this);" onselect="nextplease.enableDisableRemoveButton(this);" onchange="nextplease.syncListboxToPref(this);"/>
                    <listbox id="Last_Image_list" seltype="multiple" flex="1"
                        onkeypress="nextplease.removeSelectedOnDelete(event, this);" onselect="nextplease.enableDisableRemoveButton(this);" onchange="nextplease.syncListboxToPref(this);"/>
                </deck>
                <hbox id="images_dummy_texts" collapsed="true">
                    <textbox id="Next_Image_dummy_text" preference="nextimages" onchange="nextplease.syncPrefToListbox(this);"/>
                    <textbox id="Prev_Image_dummy_text" preference="previmages" onchange="nextplease.syncPrefToListbox(this);"/>
                    <textbox id="First_Image_dummy_text" preference="firstimages" onchange="nextplease.syncPrefToListbox(this);"/>
                    <textbox id="Last_Image_dummy_text" preference="lastimages" onchange="nextplease.syncPrefToListbox(this);"/>
                </hbox>
                <separator class="thin"/>
                <hbox align="stretch">
                    <textbox type="text" maxlength="256" onkeypress="nextplease.addOnReturn(event, this);"/>
                    <button label="&options.add;" style="margin-left: 0"
                        oncommand="nextplease.addToListbox(this);"/>
                    <spacer flex="1" minwidth="15"/>
                    <button label="&options.removeSelected;" disabled="true" style="margin-right: 2px"
                        oncommand="nextplease.removeSelectedFromListbox(this);"/>
                    <spacer flex="1" minwidth="40"/>
                    <button label="&options.restoreDefault;"
                        oncommand="nextplease.restoreDefaultListbox(this);"/>
                </hbox>
            </vbox>
        </hbox>
    </prefpane>

    <prefpane id="nextplease.debug" label="&options.debug.title;" image="chrome://nextplease/skin/Settings.png">
        <preferences>
            <preference id="log" name="nextplease.log" type="bool"/>
            <preference id="log.detailed" name="nextplease.log.detailed" type="bool"/>
            <preference id="log.file" name="nextplease.log.file" type="bool"/>
            <preference id="highlight" name="nextplease.highlight" type="bool"/>
            <preference id="highlight.color" name="nextplease.highlight.color" type="string"/>
            <preference id="highlight.prefetched" name="nextplease.highlight.prefetched" type="bool"/>
            <preference id="highlight.prefetched.color" name="nextplease.highlight.prefetched.color" type="string"/>
        </preferences>
        <vbox>
            <checkbox label="&options.log.normal;" preference="log"/>
            <checkbox label="&options.log.detailed;" preference="log.detailed"/>
            <!--<checkbox id="nextplease.log.file" label="&options.log.file;" preference="log.file/>-->
            <separator/>
            <hbox>
                <checkbox label="&options.highlight;"
                          preference="highlight"
                          oncommand="nextplease.enableDisableHighlightColorPickers();"/>
                <colorpicker type="button" preference="highlight.color"/>
            </hbox>
            <hbox>
                <checkbox label="&options.highlight.prefetched;"
                          preference="highlight.prefetched"
                          oncommand="nextplease.enableDisableHighlightColorPickers();"/>
                <colorpicker type="button" preference="highlight.prefetched.color"/>
            </hbox>
        </vbox>
    </prefpane>
</prefwindow>

Answer 1

是的，我同意这是一个错误。当第一个窗格加载时，预置窗口会根据其内容自行调整大小。就您而言，这包括几个“背景”窗格。不幸的是，检测这一点的代码在动画情况下被绕过。当前的代码看起来像这样：

if (animation) {
  if (switching)
    animate();
} else {
  if (panes > 1)
    fixHeight();
}

它应该看起来像这样：

if (animation && switching)
  animate();
else if (panes > 1)
  fixHeight();

即使启用了动画，这也会修复第一个窗格的高度。 （如果只有一个窗格，那么当然没什么可做的。）