Closed
Bug 397787
Opened 17 years ago
Closed 17 years ago
Mailing List dialogs should have ids specified
Categories
(MailNews Core :: Address Book, defect)
MailNews Core
Address Book
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
3.11 KB,
patch
|
neil
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
Currently the mailing list dialogs haven't got any ids specified. SeaMonkey is getting a new icon set for its windows, and I'm hoping to get one for the mailing list dialog. Hence we need an id. I also want to keep TB and SM in sync, so attaching a patch for both. I've gone with abListWindow as it follows on from the card which is currently abCardWindow.
Attachment #282569 -
Flags: superreview?(mscott)
Attachment #282569 -
Flags: review?(neil)
Comment 1•17 years ago
|
||
Comment on attachment 282569 [details] [diff] [review] Specify ids for mailing list dialogs Should this be ablistWindow or should abcardWindow be given a capital 'C'?
Attachment #282569 -
Flags: review?(neil) → review+
Assignee | ||
Comment 2•17 years ago
|
||
(In reply to comment #1) > (From update of attachment 282569 [details] [diff] [review]) > Should this be ablistWindow or should abcardWindow be given a capital 'C'? Let's make it ablistWindow and leave abcardWindow alone. Then we don't have to worry about the existing window icons.
Updated•17 years ago
|
Attachment #282569 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 3•17 years ago
|
||
Patch checked in with ablistWindow -> fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•