24721 - Multi addr header widget doesn't toggle after a few times.

Status: VERIFIED FIXED

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

Description

[laurel]

Using jan21 builds on NT 4.0, linux rh6.0 and mac OS 8.5.1

The widget which appears in a displayed message where the are more than a couple
addressees doesn't toggle more than a few times if the addressee list spans more
than one line when displayed in long form.  Once the widget is in the state
where it won't toggle, clicking it will result in clicking the mailto
link/header and will open a compose window accordingly. 

1.  Launch 5.0 and go to mail, login to mail account with proper password.
2.  Select a message in mail folder which has several addressees in one type of
address header (i.e. a message which has 5 email addresses in the To: or 5 email
addresses in the cc:) . 
3.  Click on the triangle widget appearing at the end of the To:, CC: or other
header line which has been collapsed to a few displayed addressee headers.  This
should cause the full list of address headers to display.
4.  Click on the triangle widget again to collapse the header line again.
5.  Repeat a couple time to toggle the addressee header line collapsed/expanded
state.

Actual Result:  After a few times you can't toggle any more. Click on widget
will translate to clicking on header/mailto link and compose window will open.
(subsequently you will probably hit another bug #23782).

Expected Result:  Should be able to toggle as many times as you wish.

Comment 1

[Phil Peterson]

I think Scott has a bug on this code already. Reassigning to him, cc rhp.
Marking M14, but not beta stopper.

Comment 2

[Scott MacGregor]

Adding hyatt to the cc list.

 I'm using visiblity-collapse to hide the long email address header view. I show
the long header view, then  toggle back to the short view and use
visiblity-collapse to hide the long email addresses again, the actual content is
still there. i.e. visiblity-collapse isn't really collapsing the content. It's
hidden all right, but if I guess where the titled button for an email address in
the long view should be and I click on it, sure enough the button gets clicked
as if it were visible.

I hope my explanation makes sense. This may end up going to XP-toolkit. I'll
play around with visiblity-collapse a little more first.

Comment 3

[David Hyatt]

Sounds like this may be evaughan's bug.

Comment 4

[Scott MacGregor]

Apparently the problem with dynamic overlays that broke this was fixed by
waterson yesterday. so this should be fixed in today's builds.

Comment 5

[Scott MacGregor]

I've been trying to find this bug all morning long! I thought I had hallucinated
it's existence. And here I accidentally marked it fixed. The comments I put
above about dynamic overlays were for another bug I must have added them here by
mistake.

re-opening

Comment 6

[Scott MacGregor]

This bug is a duplicate of Bug #24543 which is PDT+ and beta1. However, this bug
has a better explanation of the problem and what's causing it. So I'm going to
transfer the PDT+ status to this bug report and mark the other one a duplicate
of this.

Re-assiging to evaughan per hyatt's comments.

Eric, the problem is that I'm using visibility collapse to hide a div inside of
a box in the message pane. The div consists of a list of titled buttons
corresponding to each email address. The content is hidden when I apply the
visibility-collapse style. However, if I "guess" where one of the email
addresses should be and click on that spot, I'll get the menu popup associated
with that titled button.

Comment 7

[Scott MacGregor]

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

Comment 8

[Scott MacGregor]

Jar actually crashes on Win98 when he clicks on the empty space where an email
address would be if it wasn't currently hidden. This is why [crash] appears in
the key word field.

Comment 9

[Eric Vaughan]

Fixed.

Comment 10

[laurel]

I still am not able to toggle the address header widget after a few times if the
expanded list spans two lines; original problem still exists.  When I can no
longer toggle, there is a popup menu with choices to Send mail to, Add to Addr
Book or Send and Instant Message.

Reopening.  I found NT worked a little better than my linux; I could toggle a
few more times on NT before hitting the problem condition.

Using:
2000-02-10-15m14 commercial build on linux rh 6.0
2000-02-10-14m14 commercial build on NT 4.0 
having trouble with my mac, will retest that platform shortly.

Comment 11

[Peter Trudelle]

Removing PDT+ for reconsideration. We knowingly shipped 4.5 with at least one
platform only able to toggle this control _once_, and I don't recall anyone
being too broken up about it.  If anyone needs to toggle it more than once for
some strange edge-case reason, then there is a simple workaround (select another
message, then this message again).  This scenario is so remote as to barely be
worth release-noting for beta. It is not worth delaying the release, ignoring
other defects and risking regressions for.

As to the 'crash', neither this bug nor 24543 includes steps to reproduce a
crash, and in fact using current comm opt build on Win98, and an hour-old debug
tip build on Linux, I can toggle the control many times without any problems.
I'd resolve as WFM, but I don't have NT.

Comment 12

[Scott MacGregor]

Hmmm I do this all the time. It's really been a small usability issue for me.
Maybe I'm alone in this...

Comment 13

[leger]

Putting on PDT- radar for beta1.

Comment 14

[Eric Vaughan]

Have a fix testing

Comment 15

[Eric Vaughan]

fixed

Comment 16

[laurel]

OK using:
2000-03-06-09 commercial build on linux rh6.0
2000-03-06-09 commercial build on NT 4.0

Comment 17

[laurel]

OK using 2000-03-06-08 mozilla build on mac os 9.0