456863 - New/Edit Card tooltip not resetting in Header bar display

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect, P1)

Description

[Ronald Killmer]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.17) Gecko/20080829 Firefox/2.0.0.17
Build Identifier: Shredder nightly 2008/09/24

From https://bugzilla.mozilla.org/show_bug.cgi?id=450724#c79

Found a bug in the New/Edit Card Inline feature set. The tooltip is not resetting.

After viewing a Signed message in Inbox I switched to Local > Drafts to look at
how the Star worked there. On hovering the From: I saw the Tooltip still
showing the info from the prior message viewed.

I switched to a news account and hovered the posters display name in the header
view.  It also was showing the tooltip from the first viewed message. The referenced screen shot shows this.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.


Expected Results:  
Tooltip will match the full "DisplayName<email@name.x>" the cursor is hovering over in the Header Bar display.

See screen shot attachment: 
https://bugzilla.mozilla.org/attachment.cgi?id=340185

Comment 1

[Ronald Killmer]

Nominate Blocking Tb 3.0

Comment 2

[Mark Banner (:standard8)]

Attachment: The fix

I miss read this bug when I first saw it. It is especially noticeable in the collapsed view.

To repeat it in simple steps, select an email where the from address is in your address book, then select an email where the from address isn't in your address book. Then hover over and read the tooltip - it has stuck with the email address that was in your address book.

This patch fixes that by reverting back to using tooltiptext rather than tooltipemail. I'd originally separated them because I wanted a separate tooltip on the star (tooltipstar) however, the updateEmailAddressNode function in mailWidgets.xml was still using the old tooltiptext - this was causing the problem as the tooltip wasn't refreshed in the right place.

The tooltip for the star still works, I expect that is because the inherited xul attribute overrides the normal one, via the tooltiptext=tooltipstar.

Comment 3

[Mark Banner (:standard8)]

Attachment: The fix v2

Slightly updated patch that addresses a couple of other mistakes as well.

One was in UpdateEmailNodeDetails we were setting the wrong tooltip for the star if an email no longer had a card associated with it. This can be seen if you delete a card whilst display an email with which it is associated.

The other was in updateExtraAddressProcessing in mailWidgets.xml, in the for loop I had assumed that the number of email addresses (mAddresses.length) was the same as 2 * childNodes. This wasn't true. mAddresses contains all the current addresses for the element actually displayed, whereas childNodes can contain displayed email addresses as well as cached ones. So here we should be using mAddresses.length. This avoids assertions like "aAddressData is not defined" in msgHdrViewOverlay.js

To reproduce the second point you'd need to select an email with lots of to addresses, then select an email with only one email address.

Comment 4

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 340410 [details] [diff] [review]
The fix v2

Looks good; r=dmose.

Comment 5

[Mark Banner (:standard8)]

Patch checked in, changeset id 439:d0c6b8e99b06