The default bug view has changed. See this FAQ.

A character is deleted on HTML Editor when running IME reconvertion with non-selected text

VERIFIED FIXED in mozilla1.8.1

Status

()

Core
Editor
VERIFIED FIXED
12 years ago
7 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

(5 keywords)

Trunk
mozilla1.8.1
inputmethod, intl, jp-critical, verified1.8.0.2, verified1.8.1
Points:
---
Bug Flags:
blocking1.8.0.2 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tjp-dl])

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
On HTML editor, if we try to reconvert by IME when the text is not selected, the next character from the caret is deleted. It seems we pressed delete key.
This is not reproduced on Text editor.
(Assignee)

Comment 1

12 years ago
Created attachment 204171 [details] [diff] [review]
Patch rv1.0

We have a bug in nsHTMLEditRules::WillDeleteSelection.
If aAction is nsIEditor::eNone, the function is running as with nsIEditor::ePrevious. Because the process is checking aAction is nsIEditor::eNext or not. So, we should return in early time if aAction is nsIEditor::eNone.

And we don't need to run nsPlaintextEditor::DeleteSelection if the selection text is empty.

And we don't need IME code in nsHTMLEditor. Becuase nsHTMLEditor::SetCompositionString and nsHTMLEditor::GetReconversionString are same as nsPlaintextEditor's methods.(nsHTMLEditor's methods are overriding nsPlaintextEditor's one.) These code are duplicated by bug 66290. We should have removed these methods in this time.
Attachment #204171 - Flags: review?(timeless)
(Assignee)

Updated

12 years ago
Status: NEW → ASSIGNED

Updated

12 years ago
Attachment #204171 - Flags: review?(timeless) → review+
(Assignee)

Updated

12 years ago
Attachment #204171 - Flags: superreview?(bzbarsky)
I'm sorry, but this needs review or at least module owner approval from someone resembling an editor peer before I can sr it.
(Assignee)

Comment 3

12 years ago
boris:

What? This patch was already reviewed by timeless. He is editor peer.
Doh!  I missed his name in the list...

I'll try to get to this before as soon as I can, but I really don't know anything about this code (which means I'll have to learn it) and I've already got a huge backlog of reviews to do before this one, so I don't know that I'll be able to do it soon enough (that is, before I lose net access; if I don't get to it before then, then it won't happen till 2006).  No matter what, I won't be able to sr this for at least a week, which means things will be _really_ tight as far as getting it done in time...

If at all possible, I would recommend seeking another sr, preferably one familiar with IME or editor code (Simon Fraser, perhaps?)
(Assignee)

Comment 5

12 years ago
Comment on attachment 204171 [details] [diff] [review]
Patch rv1.0

Boris:

Thanks, I'm changing to Simon Fraser.
Attachment #204171 - Flags: superreview?(bzbarsky) → superreview?(sfraser_bugs)
(Assignee)

Comment 6

12 years ago
Simon:

This patch fixes bug 296686 too. That bug is very serious usability issue of Japanese EGBRIDGE users. Please sr it.
Flags: blocking1.8.1?

Updated

12 years ago
Attachment #204171 - Flags: superreview?(sfraser_bugs) → superreview+
(Assignee)

Comment 7

12 years ago
checked-in to Trunk, Thanks!
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

11 years ago
Comment on attachment 204171 [details] [diff] [review]
Patch rv1.0

This bug dosen't have any regression reports. So this patch has low risk.

But the fixed problem is serious for Mac EGBRIDGE(IME of Ja) users.
# When the users hit space key between inputted characters, the next character of caret is deleted by this bug.

Let's go to 1.8.1 and 1.8.0.1.
Attachment #204171 - Flags: approval1.8.1?
Attachment #204171 - Flags: approval1.8.0.1?
Comment on attachment 204171 [details] [diff] [review]
Patch rv1.0

Too late for 1.8.0.1, but thanks for flagging it so we can consider it for .0.2
Attachment #204171 - Flags: approval1.8.0.2?
Attachment #204171 - Flags: approval1.8.0.1?
Attachment #204171 - Flags: approval1.8.0.1-
(Assignee)

Updated

11 years ago
Attachment #204171 - Flags: branch-1.8.1?(timeless)

Updated

11 years ago
Attachment #204171 - Flags: branch-1.8.1?(timeless) → branch-1.8.1+
Attachment #204171 - Flags: approval1.8.1?
(Assignee)

Updated

11 years ago
Flags: blocking1.8.1? → blocking1.8.0.2?
Keywords: fixed1.8.1
Target Milestone: mozilla1.9alpha → mozilla1.8.1
(Assignee)

Updated

11 years ago
Keywords: jp-critical
Flags: blocking1.8.0.2? → blocking1.8.0.2+
Comment on attachment 204171 [details] [diff] [review]
Patch rv1.0

approved for 1.8.0 branch, a=dveditz for drivers
Attachment #204171 - Flags: approval1.8.0.2? → approval1.8.0.2+
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.0.2
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1 → verified1.8.1
(Assignee)

Comment 11

11 years ago
-> v.
Status: RESOLVED → VERIFIED

Updated

11 years ago
Whiteboard: [tjp-dl]
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.0.2 → verified1.8.0.2
(Assignee)

Updated

7 years ago
Keywords: inputmethod
You need to log in before you can comment on or make changes to this bug.