Closed Bug 37560 Opened 24 years ago Closed 24 years ago

nsHTMLEditor::InsertHTML needs to reformat to handle OS newlines.

Categories

(Core :: DOM: Editor, defect, P3)

Product:
Core
Component:
DOM: Editor
Platform:
All
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: cmanske, Assigned: akkzilla)

Details

Attachments

(1 file)

Patch to fix the problem
4.25 KB, patch
Details | Diff | Splinter Review 
When "pasting" html in the document, platform-specific newline conventions
need to be handled.
Attaching a patch.  Charley, can you review this?  It also includes a couple of
warning fixes in nsHTMLEditor.cpp.
Status: NEW → ASSIGNED
Target Milestone: --- → M16

    
    

    
  
This fix looks fine with me. As to the warnings, those look good too, 
except I think:
static int SetSelectionAroundHeadChildren...
should be
static nsresult SetSelectionAroundHeadChildren...
just for good form.


    
    

    
  
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED

    
    

    
  
verified in 5/8 build.
Status: RESOLVED → VERIFIED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: