13073 - [Tree] Double-clicking adds twisty even though no children

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P2)

Description

[lchiang]

Double-clicking on server name in folder pane adds a twistie when there are no
children

1999090208 release build
Win32
Setup:  You have a news server which don't have any subscribed newsgroups (ie.
top level with no children)

1) Start Messenger
2) Double click on the news server in the folder pane
3) You will see the twistie appear with a -

This should not occur.  There are no children to expand or collapse.

Comment 1

[Peter Trudelle]

targetting p3 for m14

Comment 2

[Peter Trudelle]

reassiging to putterman per hyatt, sounds like bad style rules.

Comment 3

[Alec Flett]

Taking this tree bug. (and shortening the description)

Comment 4

[Alec Flett]

Triage tree bugs. These to M13

Comment 5

[Alec Flett]

This looks like either the nsRDFGenericBuilder is adding the wrong attribute
(such as "iscontainer") or the CSS is broken.

Comment 6

[Karen Huang]

Change QA contact to me and Cc: Lisa.

Comment 7

[scottputterman]

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

Comment 8

[Richard Zach]

Note: The same thing happens in the Address Books pane.  Double click, or click
to the left of the address book icon, and you get a downward triangle.

Comment 9

[scottputterman]

I'm pretty sure I know the style rule that is causing this but I need to finish
my build before I can try it out.

Comment 10

[scottputterman]

Hyatt, can you review this checkin since it's in global.css?  Basically I
changed the style rule to check if the container is empty before putting on a
twisty.  I will add the diff after this.

Comment 11

[scottputterman]

Attachment: diff for this bug.

Comment 12

[David Hyatt]

cool.

Comment 13

[scottputterman]

I checked in the fix for this.

Comment 14

[scottputterman]

After I've checked this in, I'm wondering if maybe the real problem is that the
tree shouldn't be in this state in the first place.  That maybe when you open a
tree, it shouldn't open if it's empty?  Right now my solution is to go through
all css files and make sure that rules with [open="true] also have
[empty="false"].  But if they could never get into this state, perhaps that
would be better?

Comment 15

[Karen Huang]

Verified on 12-15-08/09-M12 all platforms.
The twistie not appear with - anymore.
Mark as verified.