Closed
Bug 345243
Opened 18 years ago
Closed 17 years ago
Missing ID "threadTreeCols" for columns of "threadTree" in file "messenger.xul"
Categories
(Thunderbird :: General, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 348504
People
(Reporter: seerdecker, Assigned: mscott)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
mscott
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.4) Gecko/20060615 Firefox/1.5.0.4
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.4) Gecko/20060615 Firefox/1.5.0.4
The tree "threadTree" defined in messenger.xul does not set an ID for its treecols element. Consequently, it is hard to write an extension that adds a new column in the message list. Several people have stumbled on this problem. The following message describes the problem in greater details:
http://www.xulplanet.com/forum/viewtopic.php?t=2081
Reproducible: Always
David Bienvenu might be interested about this bug, since he provided extensive information about doing what I'm trying to do.
Reporter | ||
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
You need to patch against cvs using `cvs diff'. I would also suggest using the '-u8p' options. Also, you need to request review for the patch. I'd suggets Scott MacGregor (mscott@mozilla.org).
Reporter | ||
Comment 3•18 years ago
|
||
Attachment #229896 -
Attachment is obsolete: true
Attachment #229975 -
Flags: review?(mscott)
Assignee | ||
Comment 4•18 years ago
|
||
Comment on attachment 229975 [details] [diff] [review]
Remade patch as suggested in comment #2
Thanks for the patch. And props to Adam for fostering this along.
Let me know if you need someone to check this in.
Attachment #229975 -
Flags: superreview+
Attachment #229975 -
Flags: review?(mscott)
Attachment #229975 -
Flags: review+
Reporter | ||
Comment 5•18 years ago
|
||
> Let me know if you need someone to check this in.
I'd appreciate it.
Thank you!
Comment 6•17 years ago
|
||
Seems the patch never made it in, but a similar fix was included by bug 348504.
Status: UNCONFIRMED → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•