Closed
Bug 71958
Opened 24 years ago
Closed 24 years ago
To: or CC: headers with multiple recipients are displayed incorrectly
Categories
(SeaMonkey :: MailNews: Message Display, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.8.1
People
(Reporter: blizzard, Assigned: hyatt)
References
Details
(Keywords: regression, Whiteboard: [nsbeta1+]critical to mozilla 0.8.1 - have patch w/reviews ready to go)
Attachments
(4 files)
|
14.51 KB,
image/jpeg
|
Details | |
|
14.51 KB,
image/jpeg
|
Details | |
|
3.25 KB,
patch
|
Details | Diff | Splinter Review | |
|
531 bytes,
patch
|
Details | Diff | Splinter Review |
screenshot
If you get a piece of email with a To: or CC: have the commas displayed in the
wrong place. I'll attach a screenshot in a second.
|
Reporter | |
Comment 1•24 years ago
|
||
|
|
Reporter | |
Comment 2•24 years ago
|
||
|
|
Reporter | |
Comment 3•24 years ago
|
||
OK, file uploading is screwed so I can't upload the entire image but if you look
you can see how the commas are before the list of recipeints. I seem to
remember some header parsing changes recently.
Whiteboard: critical to mozilla 0.8.1
|
||
Comment 5•24 years ago
|
||
same on Win32 2001-0314-04
Also noticed in mar13 commercial trunk builds.
Keywords: nsbeta1,
regression
QA Contact: esther → laurel
|
||
Comment 8•24 years ago
|
||
moving info from dup bug I filed and reassigning to racham.
Assignee: sspitzer → racham
Whiteboard: critical to mozilla 0.8.1 → [nsbeta1+]critical to mozilla 0.8.1
Target Milestone: --- → mozilla0.8.1
|
|
||
Updated•24 years ago
|
Priority: -- → P1
Hyatt thinks he probably caused this with his latest checkins. Reassigning to
him and adding myself to the cc list.
Assignee: racham → hyatt
Status: ASSIGNED → NEW
|
Assignee | |
Comment 10•24 years ago
|
||
|
|
Assignee | |
Comment 11•24 years ago
|
||
|
|
Assignee | |
Comment 12•24 years ago
|
||
Let me take this opportunity to complain about the need for *content* CSS. Not
all CSS is skin-related! In particular, rules involving display and visibility
should be loaded from content CSS files (just as we do with xul.css, which is in
content not skin).
I had to make the same change in five different skin files! All the rules were
the same! Refactor this into your content and don't force every skin to
duplicate these rules!
:)
Status: NEW → ASSIGNED
|
||
Comment 13•24 years ago
|
||
sr=mscott
|
||
Comment 14•24 years ago
|
||
r=sspitzer
|
||
Comment 15•24 years ago
|
||
Applied patch to my local tree.
Things look good both on Modern and Commercial skins.
|
||
Updated•24 years ago
|
Whiteboard: [nsbeta1+]critical to mozilla 0.8.1 → [nsbeta1+]critical to mozilla 0.8.1 - have patch w/reviews ready to go
|
||
Comment 16•24 years ago
|
||
a=asa for 0.8.1 checkin on behalf of drivers.
|
|
Assignee | |
Comment 17•24 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
||
Comment 18•24 years ago
|
||
OK using apr16 commercial trunk build: linux rh6.2, mac OS 9.0, win98
Status: RESOLVED → VERIFIED
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•