Closed Bug 255236 Opened 20 years ago Closed 20 years ago

<space> in list item inserts <br> tags instead

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.8alpha3

People

(Reporter: lohphat, Assigned: peterv)

References

Details

(Keywords: regression)

Attachments

(1 file)

Fix
840 bytes, patch
bzbarsky
: review+
bzbarsky
: superreview+
shaver
: approval1.8a3+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a3) Gecko/20040811
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a3) Gecko/20040811

Since the 20040810xx win32 builds spaces can be used in list items.

Reproducible: Always
Steps to Reproduce:
1. Create either a bullet or numbered list in composer
2. try to use spacebar
3. newlines added after current line instead of spaces

Actual Results:  
wrong character inserted

Expected Results:  
added a space in the text
This is bugging me senseless.
Severity: normal → major
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking1.8a3?
Keywords: regression
This is fine in plain text compose, but anything in HTML compose exhibits it.

It was fine with the 2004-08-09-09 trunk.

Broke with the 2004-08-10-09 trunk.
Note that my build with the patch for bug 230170 is NOT showing this problem.
Specifically, we create a <br> element for each press of the spacebar.
Summary: <space> in list item causes linefeeds instead → <space> in list item inserts <br> tags instead
My previous good build was dated July 25th.
A fresh build (Linux) can't really be used to compose mails:
All spaces where I write - at end of line - are "eaten".

workaround: insert a space, and then write BEFORE that space.


Assuming I see the same bug as this one: OS=All
OS: Windows XP → All
Mine, regression caused by the patch for bug 27382.
Assignee: composer → peterv
Attached patch Fix — — Splinter Review 

    
  

        Attachment #155923 -
        Flags: superreview?(bzbarsky)

        Attachment #155923 -
        Flags: review?(bzbarsky)

    
    

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

    
    

    
  
Comment on attachment 155923 [details] [diff] [review]
Fix

Doh.  Copy/paste fun.  ;)

r+sr=bzbarsky

        Attachment #155923 -
        Flags: superreview?(bzbarsky)

        Attachment #155923 -
        Flags: superreview+

        Attachment #155923 -
        Flags: review?(bzbarsky)

        Attachment #155923 -
        Flags: review+

    
    

    
  
Comment on attachment 155923 [details] [diff] [review]
Fix

a=shaver on this bad composer regression fix for 1.8a3.

        Attachment #155923 -
        Flags: approval1.8a3+

    
    

    
  
Fix checked in on trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha3

    
    

    
  
Verified trunk fixed - w/new moz build 204081213 on w2k

    
    

    
  
*** Bug 255446 has been marked as a duplicate of this bug. ***
Blocks: 255495

    
    

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

    
  
No longer blocks: 255495

    
    

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

    
    

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

    
  
Flags: blocking1.8a3?

    
    

    
  
Verified FIXED using 2004-08-16-13, Windows XP.
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: