Closed Bug 71958 Opened 23 years ago Closed 23 years ago

To: or CC: headers with multiple recipients are displayed incorrectly

Categories

(SeaMonkey :: MailNews: Message Display, defect, P1)

Product:
SeaMonkey
Component:
MailNews: Message Display
Platform:
x86
Linux
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
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)

screenshot
14.51 KB, image/jpeg
Details
screenshot
14.51 KB, image/jpeg
Details
Patch to the skins
3.25 KB, patch
Details | Diff | Splinter Review
Patch to global
531 bytes, patch
Details | Diff | Splinter Review 
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.
Attached image screenshot — 

    
    

    
  

      
      
      
      

        Attached image
          
        screenshot
        — 
      

    


  

    
    

    
  
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

    
    

    
  
*** Bug 71964 has been marked as a duplicate of this bug. ***

    
    

    
  
same on Win32 2001-0314-04

    
    

    
  
Also noticed in mar13 commercial trunk builds.
Keywords: nsbeta1, 
          
            regression
QA Contact: esther → laurel

    
    

    
  
*** Bug 71978 has been marked as a duplicate of this bug. ***

    
    

    
  
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

    
  
Status: NEW → ASSIGNED

    
  
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

    
    

    
  


  

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Patch to global
        — 
          — Splinter Review
      

    


  

    
    

    
  
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

    
    

    
  
sr=mscott

    
    

    
  
Applied patch to my local tree.

Things look good both on Modern and Commercial skins.



    
  
Whiteboard: [nsbeta1+]critical to mozilla 0.8.1 → [nsbeta1+]critical to mozilla 0.8.1 - have patch w/reviews ready to go

    
    

    
  
a=asa for 0.8.1 checkin on behalf of drivers.  

    
    

    
  
Fixed.

Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED

    
    

    
  
OK using apr16 commercial trunk build: linux rh6.2, mac OS 9.0, win98
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: