1012925 - Undeclared panel height prevents workaround of other panel rendering bug(s)

Status: UNCONFIRMED

(Firefox :: Toolbars and Customization, defect)

Description

[Chris Tomlinson]

As mentioned in bug #994194, the height value is currently important to allow the workaround of other panel bug(s) to work. I have used this workaround in my addon and have reproduced the comment from #994194 here. I've filed a separate bug to track the sizing bug that causes the panel height to be required. After testing against FF30, some of the unpredictability of the panel height bug appears to have already been resolved so I'm hopeful that we can address this bug by fixing bug #1012920 rather than reverting bug #994194 or implementing another temporary workaround in my addon.

My addon (KeeFox[1]) has been broken by the change introduced in #994194 because I set my addon panel height to the same value as set in the property removed by #994194.

The reason I do this is that the default behaviour of the PanelUI in FF29 is to produce panels that do not have the correct size and/or scrollbar presence for the (XHTML) content my addon includes in the panel. (see bug #1012920)

By adjusting my addon panel height after the panel has been opened (and via a setTimeout) I managed to work around the problem. The workaround is in this bit of source code: https://github.com/luckyrat/KeeFox/blob/v1.4.1/Firefox%20addon/KeeFox/chrome/content/panel.js#L1601-L1627

I notice that there are a variety of related bugs being targeted at the nightly channel so I hope that FF32 will resolve the underlying problems that cause me to require this workaround. I'm just not sure how else I can work around the problem in 30 and 31 if #994194 is not reverted.

[1]: KeeFox - https://addons.mozilla.org/en-US/firefox/addon/keefox/