Open Bug 131854 Opened 23 years ago Updated 4 years ago

Cursor not always positionned correctly after nsIEditorShell::InsertSource

Categories

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

defect

Tracking

()

mozilla1.4beta

People

(Reporter: bugzilla, Unassigned)

Details

While working on bug 107425, I found that when I call nsIEditorShell::InsertSource, the cursor was not always correctly positionned at the end of the newly inserted data.
jfrancis, from reading the bug 107425, it sounds like when we are done, we are collapsing the caret to the point where we did the insert instead of collapsing after it. I don't think you can use mail to reproduce this anymore since the fix for bug 107425 looks like the caret is manually moved to the end of the doc after their insertion which I think is the right thing for them to be doing anyways since we can change our "caret placement after edit operation" policies at any time ... your recent place the caret in the last item in pasted content change comes to mind. In any case, for consistency sake, I think the InsertSource() should probably follow Paste's collapsing model. If we did that, mail would've had to do their fix for bug 107425 anyways.
OS: Windows 2000 → All
Hardware: PC → All
Doh, it looks like I forgot to reassign this to jfrancis, back in March. Joe, see my comments above.
Assignee: kin → jfrancis
Target Milestone: --- → mozilla1.1alpha
The trunk is the wave of the future!
Target Milestone: mozilla1.1alpha → mozilla1.1beta
Target Milestone: mozilla1.1beta → mozilla1.2beta
The days of having a half dozen milestones out in front of us to divide bugs between seem to be gone, though I dont know why. Lumping everything together as far out as I can. I'll pull back things that I am working on as I go.
migrating milestones now that there are more available. i'll pull these back as I work on them
Target Milestone: mozilla1.2beta → mozilla1.4beta
QA Contact: sujay → editor
Assignee: mozeditor → nobody

Bulk-downgrade of unassigned, untouched DOM/Storage bug's priority.

If you have reason to believe, this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.