Closed Bug 188288 Opened 22 years ago Closed 21 years ago

[patch] caret movement erratic with CTL enabled

Categories

(Core :: Layout: Text and Fonts, defect)

x86
Linux
defect
Not set
major

Tracking

()

RESOLVED FIXED

People

(Reporter: Louie.Zhao, Assigned: prabhat.hegde)

References

Details

(Keywords: intl)

Attachments

(2 files, 3 obsolete files)

mozilla (both gtk and gtk2 build) with "--enable-ctl" will cause caret to behavior disorderly.It's quite severe since it always happen when user compose a mail. Reproduce steps: 1. open mailnews and compose a mail using html format 2. type a paragraph without hit return which can be automaticly wrapped into more than one line. 3. use left arrow key to move back expect, the caret moves back result, the caret runs to some unexpect place ( for example, directly to the behind of the 5th charact of the first line)
Keywords: intl
This is due to an error in logic of: http://lxr.mozilla.org/seamonkey/source/intl/ctl/src/nsULE.cpp#305. Here is what i am planning to do. Use the basic-shaper to handle all scripts other than CTL in pangolite. This will clean up the logic in handling caret positions. This will require some re-write and testing of nsULE and also addition of new shaper but should be done by next week. prabhat
Attached patch Patch to fix this problem (obsolete) (deleted) — Splinter Review
The patch does the following: A> Fixes a logical bug in CTL cursor placement logic. B> Prevents CTL cursor positioning logic from being used when the endpoints of an edit operation do not require CTL processing. Whether or not a script requires Complex processing is decided based on if it has a pangolite shaper. This fix also has the side effect of speeding up non-CTL cursor positioning. Note that this patch sits on top of patch for bugzilla #184599.
*** This bug has been marked as a duplicate of 122879 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
Prabhat Hegde: Should we reopen this bug or move your patch to bug 122879 ?
Reopening bug since this one has a patch...
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment on attachment 115149 [details] [diff] [review] Patch to fix this problem prabhat: The patch is slightly rotten - can you try to update it to "trunk", please ?
Blocks: 122879
Attached patch patch updated (obsolete) (deleted) — Splinter Review
I updated attachment 115149 [details] [diff] [review], got rid of tabs and cleaned up other white spaces.
Attachment #115149 - Attachment is obsolete: true
*** Bug 178735 has been marked as a duplicate of this bug. ***
let's move it to layout/ctl for a better tracking. I18N can be tracked with 'intl' keyword.
Blocks: 201746
Component: Internationalization → Layout: CTL
Summary: the move of caret by arrow keys is disordered due to "--enable-ctl" → [patch] caret movement erratic with CTL enabled
Attachment #138248 - Flags: superreview?(rbs)
Attachment #138248 - Flags: review?(prabhat.hegde)
*** Bug 176272 has been marked as a duplicate of this bug. ***
selection (Bug 100173) and highlighting (Bug 167983) also broken
Attached patch udpdated patch (obsolete) (deleted) — Splinter Review
There was a typo in the previous patch I didn't notice because of the way I compiled. Anyway, with this patch, it seems to work fine for non-complex scripts. I have to hunt for an Xkb map for Devanagari or Thai to test it fully. Here, I also removed 'm'-prefix used for local variables. Mozilla-convention (not everywhere but certainly in nsTextFrame.cpp) is to use 'm' prefix for member variables only.
Attachment #138248 - Attachment is obsolete: true
Comment on attachment 138248 [details] [diff] [review] patch updated .
Attachment #138248 - Flags: superreview?(rbs)
Attachment #138248 - Flags: review?(prabhat.hegde)
So did this patch make it into the Solaris build of 1.6, or do I need to build my own without --enable-ctl?
Comment on attachment 138278 [details] [diff] [review] udpdated patch Requesting r=/sr= to get some progress here. Rumor says we have little more than a week until 1.7a and I'd like to see this problem fixed for that release.
Attachment #138278 - Flags: superreview?(dbaron)
Attachment #138278 - Flags: review?(prabhat.hegde)
Could you attach a diff -ub as well? (Ignores indentation changes.)
Flags: blocking1.7b?
Attachment #138278 - Flags: superreview?(dbaron)
Attachment #138278 - Flags: review?(prabhat.hegde)
Attachment #138278 - Attachment is obsolete: true
Comment on attachment 141978 [details] [diff] [review] Updated patch for 2004-02-22-trunk r=roland.mainz@nrubsig.org (original patch was from prabhat). The patch compiles and fixes the problem... :)
Attachment #141978 - Flags: superreview?(bzbarsky)
Attachment #141978 - Flags: review+
Attachment #141978 - Flags: superreview?(bzbarsky) → superreview?(rbs)
(In reply to comment #14) > So did this patch make it into the Solaris build of 1.6, or do I need to > build my own without --enable-ctl? No, I am afraid the bug is still there in 1.6 and you will have to compile it yourself. The standard 1.6 build for sparc-sun-solaris2.8 is built with "--disable-tests --disable-debug --with-xprint --enable-xinerama --enable-ldap --enable-crypto --enable-x11-shm --enable-ctl" . Kind regards, Nils.
Attachment #141978 - Flags: superreview?(rbs) → superreview?(roc)
Comment on attachment 141978 [details] [diff] [review] Updated patch for 2004-02-22-trunk + static NS_DEFINE_CID(kLECID, NS_ULE_CID); These should really be hoisted to file scope. sr=roc whether you do that or not
Attachment #141978 - Flags: superreview?(roc) → superreview+
Checking in layout/html/base/src/nsTextFrame.cpp; /cvsroot/mozilla/layout/html/base/src/nsTextFrame.cpp,v <-- nsTextFrame.cpp new revision: 1.453; previous revision: 1.452 done (with the DEFINE_CID moved directly after the #include in the ifdef SUNCTL block)
Status: REOPENED → RESOLVED
Closed: 22 years ago21 years ago
Resolution: --- → FIXED
I have redone the Mozilla 1.7a release builds for Solaris with this patch applied: MD5 (http://ftp.mozilla.org/pub/mozilla.org/mozilla/releases/mozilla1.7a/contrib/mozilla-sparc-sun-solaris2.8-1.7a.tar.gz) = 3174bae0d6e4a22d098936ae67505264 MD5 (http://ftp.mozilla.org/pub/mozilla.org/mozilla/releases/mozilla1.7a/contrib/mozilla-sparc-sun-solaris2.8-gtk2-1.7a.tar.gz) = a67554ba5f61c1c2c11788545046a52d Can anyone please verify that the issue is fixed in these builds ?
(In reply to comment #23) > mozilla-sparc-sun-solaris2.8-gtk2-1.7a.tar.gz) > Can anyone please verify that the issue is fixed in these builds ? Just tested it a little, but it seems fine. It was quite easy to reproduce before.
Andreas Lange wrote: > > mozilla-sparc-sun-solaris2.8-gtk2-1.7a.tar.gz) > > Can anyone please verify that the issue is fixed in these builds ? > > Just tested it a little, but it seems fine. It was quite easy to reproduce > before. OK... should I respin the mozilla1.6 release builds incl. this patch, too ?
I just tested the build from Roland (see comment #23) The erratic behavior of the carret seems to be gone in mailnews composer windows and in HTML form TEXTAREA boxes. But I encountered another problem, for which I'm not sure, if it's a problem of the bug fix or of 1.7a. Copying text from an CDE window (dtterm Terminal or dtpad Editor) doesn't work any more. Here are some steps to reproduce that problem: - open a mailnews composer window (I use the plan ascii version) - mark some text from a CDE dtterm - try to copy the marked text by pressing the middle button into the composer window Result: text is not copied. You can try it by using the "Edit --> Copy" and "Edit --> Paste" functions in both dtterm and mozilla, but that has the same effect. Copying text between two mozilla windows works as expected. Can somebody please verify this? Is this a known bug in 1.7a? (I kept 1.6 until now) I'm currently sitting at a SunRay terminal connected to a Solaris 9 box, but I don't think this could be the problem. I'm also currently applying the recommended patches for mozilla to a box with Solaris 8 and retry the test. I'll try the original mozilla 1.7a build for solaris on that box, too.
Meik Langwald wrote: > I just tested the build from Roland (see comment #23) > > The erratic behavior of the carret seems to be gone in mailnews composer > windows and in HTML form TEXTAREA boxes. > > But I encountered another problem, for which I'm not sure, if it's a problem > of the bug fix or of 1.7a. Copying text from an CDE window (dtterm Terminal or > dtpad Editor) doesn't work any more. That issue is not related to CTL(=Complex Text Layout) ... what you're seeing is bug 233317 ("Cannot paste from clipboard") ... ;-(
Flags: blocking1.7b?
Component: Layout: CTL → Layout: Text
QA Contact: amyy → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: