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

VERIFIED FIXED in M16

Status

()

Core
Editor
P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: Charles Manske, Assigned: Akkana Peck)

Tracking

Trunk
All
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
When "pasting" html in the document, platform-specific newline conventions
need to be handled.
(Assignee)

Comment 1

18 years ago
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
(Assignee)

Comment 2

18 years ago
Created attachment 8091 [details] [diff] [review]
Patch to fix the problem
(Reporter)

Comment 3

18 years ago
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.
(Assignee)

Comment 4

18 years ago
Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 5

18 years ago
verified in 5/8 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.