Closed Bug 331924 Opened 19 years ago Closed 18 years ago

message header view cleanup

Categories

(Thunderbird :: General, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird2.0

People

(Reporter: mscott, Assigned: mscott)

References

(Blocks 2 open bugs)

Details

(Keywords: fixed1.8.1)

Attachments

(4 files)

Before I get cracking on custom headers for the message pane instead of view all headers, there's some code cleanup I want to do.
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird2.0
Attachment #216481 - Flags: superreview?
Comment on attachment 216481 [details] [diff] [review] use a deck to toggle between the collapsed and expanded view use a deck to control the collapsed and expanded header view. The deck does most of the work for us that we were doing in JS.
Attachment #216481 - Flags: superreview? → superreview?(bienvenu)
this starts to rip out the keyword stuff which was added for Netscape back in 2002 so they could add extra UI stuff.
Attachment #216484 - Flags: superreview?(bienvenu)
Attachment #216484 - Flags: superreview?(bienvenu) → superreview+
Attachment #216481 - Flags: superreview?(bienvenu) → superreview+
I'm not adverse to support images in the message header pane (x-face, etc.), but we won't do it by looking for netscape instant messaging file screen name paths in your profile since we don't use netscape instant messenger anymore.
Attachment #216576 - Flags: superreview?(bienvenu)
Attachment #216576 - Flags: superreview?(bienvenu) → superreview+
we turned this functionality off a long time ago, it made it too easy to get spam addresses into your address book. This just removes the code that looks at the pref all together.
Attachment #216578 - Flags: superreview?(bienvenu)
Attachment #216578 - Flags: superreview?(bienvenu) → superreview+
the keyword patch broke message header display on the 2.0 branch due to a bad merge. I just fixed that so tomorrow's builds should be all better.
Did this cause Bugzilla Bug 332267 header + not working? So far, it seems better with version 2 alpha 1 (20060331)
Depends on: 340135
I didn't have any more open items for this.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Can we get this bug fixed on branch also? I did a bug for branch, but it was duped to this one. Thanks.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: