55577 - modern--the preference dialog box messes up

Status: VERIFIED FIXED

(Core :: XUL, defect, P2)

Description

[Patty Mac]

Linux only (2000-10-06-10-MN6)

Steps to reproduce:

1. Launch Netscape 6
2. Go to Edit/Preferences/Themes
Note: The dialog box looks ugly and messes up. Only see Classic skin 
      shows up. Should able to see the Modern skin as well. So the 
      users should have a choice of switching from Modern to Classic
      or Classic to Modern.
      There is work-around by going to View/Apply Themes/
      and then repeat step #2, the dialog box looks back to normal.

Comment 1

[Patty Mac]

I nominate this bug is rtm because the user won't have a choice by changing 
from modern to classic.

Comment 2

[don]

Hmmmmm, don't see the problem with today's branch bits.  Is this on thr trunk?
Maybe you can attach a screen dump of the horkage ...

Comment 3

[Patty Mac]

Attachment: screen shot is attached

Comment 4

[Patty Mac]

Don, I tried on Linux (2000-10-12-11-MN6), the problem is still there.
Screen shot already attached. The bug is re-open.

Comment 5

[don]

Ugh.  That IS bad.  Ouch.

Comment 6

[don]

Dave, Ben sez this looks like tree widget horkage.  Could it be related to
Eric's tree widget re-write?

Comment 7

[don]

*** Bug 55092 has been marked as a duplicate of this bug. ***

Comment 8

[John Morrison]

So, Dave, I found that I could reproduce this on my machine. It can occur with 
either modern or classic, and is dependent on the width of the pref panel 
(unfortunately the default width is one of the widths where this occurs). 

I stumbled onto a hacky workaround, which we could consider for RTM. If I place 
the <tree> in a <box style="min-height:100px"> then this works fine. Don't ask 
me why that works (although I suppose it may avoid a reflow as the tree gets
built (hand-waving here)).

This has a side-effect of narrowing the tree, but not unattractively (it's 
better than the alternative). You do not want to put a flex on the tree, 
though, because then this stops working again. [Yeah, this is a real "duct 
tape" kind of workaround].

Comment 9

[Peter Trudelle]

rtm-/future.  We're supposed to be baking now, and taking only topcrash/dataloss
bugs. This is obscure, cosmetic, and has workarounds.

Comment 10

[David Hyatt]

I believe this was fixed by 57139.

Comment 11

[Patty Mac]

Marking verified on Linux (2001-01-24-08-Mtrunk).