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] Tue Apr  8 08:53:20 +0000 
2008 -------
Thanks for looking into this!

The kashida width currently used for GenericSalLayout::KashidaJustify() is just 
the logical width of an 
isolated glyph for the font's U+0640 (arabic tatweel). The idea is that it 
doesn't make sense to insert 
justification glyphs if they are too wide. Do you happen to know how the member 
iKashidaWidth from usp's FontProperties and this glyph width correlate?

I'm sure that GenericSalLayout::KashidaJustify() could benefit from more 
debugging. I'd delay this task 
though until we are happy with the justification result on one platform (USP?) 
and then port it to the 
other platforms. GenericSalLayout is mainly used on our Unix ports.

Writer does its cursor placement usually depending on the result of 
OutputDevice::GetTextArray() which 
is very related to UniscribeLayout::FillDXArray(). But when you override the 
resulting DXArray with your 
improved justification method then the cursor positioning needs to be adjusted 
too. FME can probably 
point you to the matching code in Writer.

The problem with vowel placement on a stretched letter is probably caused 
because a glyph cluster is 
not properly detected. Only glyph clusters themselves should get adjusted, the 
positioning inside a 
glyph cluster is supposed to be unchanged.

Hope this helps.

---------------------------------------------------------------------
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]

Reply via email to