Closed Bug 150915 Opened 22 years ago Closed 22 years ago

Arial is "foo" when toolbar is used to select font

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.0.1

People

(Reporter: cody, Assigned: cmanske)

References

Details

(Keywords: regression, Whiteboard: [adt2 RTM][fixed in trunk],[verified-trunk])

Attachments

(1 file, 1 obsolete file)

CVS Patch
1.20 KB, patch
cmanske
: review+
kinmoz
: superreview+
jesup
: approval+
Details | Diff | Splinter Review 
Steps to reproduce:

1. Create a new message.
2. Type some text.
3. Using the font selector *in the toolbar*, choose "Helvetica, Arial".
4. Select All.
5. View the source using the Insert -> HTML command.

Expected results:  The FONT FACE will be "Helvetica, Arial, sans-serif".

Results:  The FONT FACE is set to "Helvetica, foo, sans-serif".

Interestingly, the font attribute is correctly set if you use the Format->Font
menu instead of the toolbar to choose Helvetica.
*** Bug 150917 has been marked as a duplicate of this bug. ***
*** Bug 150917 has been marked as a duplicate of this bug. ***
I see the problem.  Patch coming up.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch Patch (obsolete) — — Splinter Review 

    
    

    
  
This is a no-risk patch that needs to also get into the branch.  Reviews needed.
Keywords: mail6, 
          
            mozilla1.0.1, 
          
            nsbeta1, 
          
            patch, 
          
            regression, 
          
            review
OS: Windows 98 → All
Hardware: PC → All

    
    

    
  

      
      
      
      

        Attached patch
          
          
        CVS Patch
        — 
          — Splinter Review
      

    


  
Here's a real patch against CVS

        Attachment #87363 -
        Attachment is obsolete: true

    
    

    
  
reassign to editor team
Assignee: ducarroz → cmanske
Component: Composition → Editor: Composer
Product: MailNews → Browser
Target Milestone: --- → M1

    
    

    
  
Comment on attachment 87407 [details] [diff] [review]
CVS Patch

r=cmanske
I wonder how the heck that happened! (CVS Blame doesn't reveal when that
changed was made.)

        Attachment #87407 -
        Flags: review+

    
    

    
  
I have some vague recollection that some Linux users didn't like Arial
Akkana: Do you know anything about this?

Status: NEW → ASSIGNED
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt2 RTM][FIX IN HAND][need sr=]

    
    

    
  
The arial problems on linux are discused in bug 46415.  Redhat had a problem up
through 7.1; I think 7.3 and later are better (my 7.3 machine doesn't have any
Arial at all).  Some other distros had problems as well, especially if they had
installed abiword or Japanese fonts (for some reason those packages come with an
awful scaled-bitmap arial font).  Listing Helvetica before Arial solves the problem.

    
    

    
  
CVS claims that the bug was introduced:
1.112 cmanske%netscape.com Aug 9 2000   Big UI fix checkin continued - same 
bugs as last checkin. r=sfraser

If that's true, that would mean there could be a large amount of people 
receiving 'foo' in their mailbox and many webpages with 'foo' in them since it 
would have been included in practically every version of Mozilla and Netscape 
(since M17).

    
    

    
  
Charley--since this isn't your patch, you may not want to be the "assigned to"
person.
Target Milestone: M1 → mozilla1.0.1

    
    

    
  
Comment on attachment 87407 [details] [diff] [review]
CVS Patch

sr=kin@netscape.com

        Attachment #87407 -
        Flags: superreview+

    
  
Whiteboard: [adt2 RTM][FIX IN HAND][need sr=] → [adt2 RTM][FIX IN HAND]

    
    

    
  
checked into trunk
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Keywords: patch, 
          
            review → adt1.0.1
Resolution: --- → FIXED
Whiteboard: [adt2 RTM][FIX IN HAND] → [adt2 RTM][fixed in trunk]

    
    

    
  
gone, verif on trunk 0617.
Whiteboard: [adt2 RTM][fixed in trunk] → [adt2 RTM][fixed in trunk],[verified-trunk]

    
    

    
  
adding adt1.0.1+.  Please get drivers approval before checking into the branch.
Keywords: adt1.0.1 → adt1.0.1+

    
    

    
  
Comment on attachment 87407 [details] [diff] [review]
CVS Patch

Approval granted for 1.0 branch checkin; please remove the mozilla1.0.1+
keyword when this is fixed and add the fixed 1.0.1 keyword.

        Attachment #87407 -
        Flags: approval+

    
  
Keywords: mozilla1.0.1 → mozilla1.0.1+

    
    

    
  
checked into mozilla1.0.1 branch
Keywords: mozilla1.0.1+ → fixed1.0.1

    
    

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

    
    

    
  
Regarding Mozilla 1.0.1 branch:  which release version will this be available
in?  I'm using 1.1a and this isn't checked in... is the next version of Mozilla
to be 1.1b or are we going back in version numbers to 1.0.1?
    One other comment...  with Windows users making up over 90% of the computer
market, shouldn't Arial be listed first (Arial, Helvetica, sans-serif)?  Arial
is a common font on Windows systems while it is rare for a Windows system to
have Helvetica.   
   On my Mac OS X system I have both Arial and Helvetica... but I prefer the
look of Arial.
  

    
    

    
  
For version number info, see http://www.mozilla.org/roadmap.html
For the reason Helvetica is listed before Arial, see bug 46415.

    
  
QA Contact: esther → sujay

    
    

    
  
verified on branch.
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.1 → verified1.0.1

    
    

    
  
*** Bug 160219 has been marked as a duplicate of this bug. ***
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: