48210 - Scrollbar too long in JS console after switching panels

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect, P3)

Description

[Adrian]

Do a few JS errors (min. 8 errors), eg. in the JS Console run the mouse over
"Privacy and Security" in the Tasks menu a few time. Now select "Warnings". The
ScrollBar will be as long, as was in the "Errors" section.

Summary: In the "Warnings" section the srollbar is too long, if in the "Errors"
section the scrollbar isn't null size.

Comment 1

[John Bandhauer]

The JS console is not actually the "JavaScript Debugger". I'm not sure what 
component it is. Nevertheless, I think Ben Goodger did the UI work.

Comment 2

[Asa Dotzler [:asa]]

setting bug status to New and moving to XP Apps

Comment 3

[Peter "jag" Annema]

Hmmm...

So the CSS is hiding the treecells, but apparantly the scrollbar doesn't take
that into account. I'm guessing the tree itself doesn't (n rows * row-height).

Comment 4

[Ben Goodger (use ben at mozilla dot org for email)]

Yes, the tree doesn't know how to recompute its height when you set items to
display: none. One solution might be to assert script errors into an RDF
datasource, then re-root the tree on different subsets of errors. Another
solution might be to just ditch the tree and use a box. 

Comment 5

[Peter "jag" Annema]

I like the RDF datasource version.

s/box/grid/ ?

Comment 6

[Paul Chen]

nav triage team:

Although nice to have, don't think we'll get to it for beta1. Marking nsbeta1-

Comment 7

[Boris Zbarsky [:bzbarsky]]

reassigning to hewitt who has agreed to take JS console bugs.
part of the "triage Most Doomed people's bugs" effort

Comment 8

[Joe Hewitt (gone)]

fixed by the jsconsole re-write