7/27 linux build: In the html mail compose window, but not in the html editor window, for each word you type, as soon as you start typing the word, the caret jumps to the next line. When you type a space to end the word, the word jumps back to the previous line where it belongs, but as soon as you type a non-space character, the next word jumps back down to the next line. It's very difficult to type this way. Cc'ing everybody who checked into editor or layout yesterday.
and setting to m17 -- can each of you please take a look and see if maybe you're the culprit. I would appreciate some type of confirmation in the bug as to your findings -- thanks
Per sujay, only on TRUNK build, not on M17 branch. sujay, is this on ALL trunk platforms or just Linux?
Whiteboard: [dogfood+][nsbeta3+] FOR TRUNK
happens on WinNT. akkana says plain & html mail compose.
OS: Linux → All
Hardware: PC → All
This is probably from my checkins to remove nsTextRun yesterday. I'll take a look.
My fix to bug 19051 was overly aggressive because it didn't take into account the fact that some (most!) frames will interrupt the flow of text. It was blindly traversing through *all* the frames contained in a block without regard for whether they could "break" a word or not. This patch makes the code look a bit more like the original code. Instead of making it the responsibility of each frame to find the text runs it contains, we now query the frame to see if it can "break a word". The only text frames that can't break a word are: - generic inline frames - first letter style frames - text frames (This list corresponds exactly to the frames that implemented nsIFrame::FindNextText to add themselves to the a text run, rather than simply ending the text run.) Update nsLineLayout::FindNextText() to check each frame as it traverses it; if it encounters a "word breaker", then it leaves the loop and lets the caller know that there is no possibility of text continuation. roc+moz, whatdya think?
Status: NEW → ASSIGNED
Whiteboard: [dogfood+][nsbeta3+] FOR TRUNK → [dogfood+][nsbeta3+] FOR TRUNK, FIX IN HAND
Created attachment 12085 [details] [diff] [review] call it CanContinueTextRun() instead of IsWordBreaker()
fix checked in, r=akkana
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
verified in 8/3 build.
Status: RESOLVED → VERIFIED
*** Bug 46815 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.