I have code that collapses richlistitems based on an attribute on the richlistbox, the problem is it's showing empty space for some of the collapsed richlistitems. This used to work just fine, and display:none still works fine as well. I'll try to get a testcase together sometime soon...

This only happens when there's a binding attached to the richlistitem and new items are appended dynamically. I can't really work up a great testcase, but I'll zip up the files involved and attach them to this bug.

STR: 1) Unzip into folder of choice and create a manifest that points to it under the name "Console". 2) Navigate to chrome://Console/content/console2.xul 3) Open a new tab and go to yahoo (to collect warnings). 4) Collect some errors (javascript:e in the location bar works). 5) Reload the console, then select the "Errors" tab. 5) Minimize window until scrollbars appear. You'll notice that there's a bunch of empty whitespace below the last error item, although nothing actually shows. getComputedStyle doesn't report it as visibility: collapse, which is bad. /me searches for a regression range.

This has been around forever, and it's a Style bug since the style isn't being computed correctly as evidenced in the previous comment with getComputedStyle. I *think* the problem might be that the type="warning" and type="error" richlistitems are bound to the same binding, maybe confusing the system? display: none works though, so I'll just have to use that.

(In reply to comment #2) > 1) Unzip into folder of choice and create a manifest that points to it under > the name "Console". I don't know what this means. Could you attach such a manifest?

Here's the manifest file that goes along with it. Notice, this is on a WIndows system, you'll have to change the path if you use it on another system.

Your steps to reproduce work for me. (I assume by "minimize the window" you mean "reduce the size of the window"... which I did in both dimensions, although maybe you meant only one.)

And by "work for me", I mean I don't see anything wrong.

And when I look in DOM inspector's computed style panel, the richlistitems in question show as visibility:collapse.

These STR reproduce the bug 100% of the time for me, 1) open a blank page 2) continually hit enter on javascript:e (collect about 10 errors) 3) open www.yahoo.com to collect some warnings 4) Navigate to chrome://console/content/console2.xul 5) Hit the "Errors" tab, make sure the window is small enough (lengthwise) to display scrollbars. You'll see at the bottom some empty space at the bottom of the richlistbox that shouldn't be there, this is on: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.3a1pre) Gecko/20090818 Minefield/3.7a1pre

Oh, also make sure you have the errors still in the console, it might get overwritten by the warnings from yahoo, in which case just use google.com, I think it has less warnings.

Uhm, actually, just checked in DOMi and it is coming up as "collapse", not sure anymore where that space is coming from... In any event it doesn't happen when I use display: none

By "length", do you mean width or height?

Height :) sorry about that...

Bug 513318 has a better testcase, duping.

Was your problem fixed by the patch in bug 513318?