To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=60594
------- Additional comments from [EMAIL PROTECTED] Wed Sep 17 15:25:21 +0000 2008 ------- @hennerdrewes: [...] Other questions / comments: I have the feeling that since I made the changes to SwBidiPortion::GetSpaceCnt(), there is a inaccuracy in the cursor placement, when the cursor is placed on the boundary between LTR and RTL runs. Are you really sure, there was no deeper intention behind the way GetSpaceCnt() was implemented? [...] No, of course I'm not ;-) Have a look at the code following this comment in itrcrsr.cxx: // we came from inside the bidi portion, we want to blink... There, we call SwBidiPortion::CalcSpacing() with rInf.nIdx already set to the start of the next portion. Now that the number of blanks in the bidi portion is calculated on-the-fly we have to make sure that rInf.nIdx is set to the start of the bidi portion, just like with a regular SwTxtPortion::GetSpaceCntnt call. I suggest to store the start of the bidi portion if we set pLastBidiPortion and set this value temporarily at rInf before calling CalcSpacing(). --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
