diff options
| author | Khaled Hosny <khaledhosny@eglug.org> | 2018-03-13 16:14:47 +0200 |
|---|---|---|
| committer | Khaled Hosny <khaledhosny@eglug.org> | 2018-03-13 18:04:29 +0100 |
| commit | ae0a3825f5ad06ff8d6ae3cb388e215d8abe2eb7 (patch) | |
| tree | c8a6c8435e22da0b9ebe38f81b861bd13f894c2c | |
| parent | 226ec5aebb65f4017c29bc3aaacfaa5782dfbbb7 (diff) | |
tdf#116344: Don’t Kashida justify Syriac text
I’ve always suspected this check in VCL is wrong since Writer calculates
Kashida positions for Arabic only. If we *really* want to Kashida
justify Syriac, then Writer has to be first taught how to calculate
Kashida positions for it.
Change-Id: I20c03a3a54bce14c9bebb55426bd22537fe50d6a
Reviewed-on: https://gerrit.libreoffice.org/51223
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
| -rw-r--r-- | vcl/source/gdi/CommonSalLayout.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/vcl/source/gdi/CommonSalLayout.cxx b/vcl/source/gdi/CommonSalLayout.cxx index de701febe63b..8c509afe3d51 100644 --- a/vcl/source/gdi/CommonSalLayout.cxx +++ b/vcl/source/gdi/CommonSalLayout.cxx @@ -723,8 +723,7 @@ bool CommonSalLayout::LayoutText(ImplLayoutArgs& rArgs) if (u_isUWhiteSpace(aChar)) nGlyphFlags |= GlyphItem::IS_SPACING; - if ((aSubRun.maScript == HB_SCRIPT_ARABIC || - aSubRun.maScript == HB_SCRIPT_SYRIAC) && + if (aSubRun.maScript == HB_SCRIPT_ARABIC && HB_DIRECTION_IS_BACKWARD(aSubRun.maDirection) && (nGlyphFlags & GlyphItem::IS_SPACING) == 0) { |