225967 - Atoms for priority is wrongly disabled

Status: NEW

(Thunderbird :: Mail Window Front End, enhancement)

Description

[Thomas Berg]

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.5) Gecko/20030925
Build Identifier: Thunderbird 0.4a 20031110

There is a need for atoms for the priority status (to make it posible to have
icons for the different priority statuses).  Unfortunately it seems to be that
these are disabled.  Looking at the source, e g
http://lxr.mozilla.org/mozilla/source/mailnews/base/src/nsMsgDBView.cpp#188 the
creation of the atoms seems to be ifdefed by "SUPPORT_PRIORITY_COLORS".

Besides removing the ifdef it seem to be a need to expand number of atoms as
from looking at:
http://lxr.mozilla.org/mozilla/source/mailnews/public/MailNewsTypes.h#75,
compared to nsMsgDBView.cpp#188 above, there is three more priority statuses
that are not addressed (notSet/none/normal).

Reproducible: Always

Steps to Reproduce:
1.
2.
3.



This seem to be a very easy change and as it also is a very visible enhancement
to the UI I think it should have a high priority (sic!).

Comment 1

[Samir L. Boulema]

Would like to see this 2. it would make a nice addition to the status themes 
out there for thunderbird.

Comment 2

[Thomas Berg]

AFAICS, this is a very easy fix (admittedly, my view is somewhat limited... :) ).

What is the problem with this ?

TIA
Thomas