Bug 114257 - DRAW: Bad "Convert - To Curve" some arabic letters from the Text Box
Summary: DRAW: Bad "Convert - To Curve" some arabic letters from the Text Box
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Draw (show other bugs)
Version:
(earliest affected)
5.4.3.2 release
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: ⁨خالد حسني⁩
URL:
Whiteboard: target:7.5.0 inReleaseNotes:7.5
Keywords: bibisected, bisected, regression
Depends on: 104921
Blocks:
  Show dependency treegraph
 
Reported: 2017-12-05 11:34 UTC by Kamil Landa
Modified: 2022-12-06 17:15 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Curves from the text (10.23 KB, application/vnd.oasis.opendocument.graphics)
2017-12-11 16:21 UTC, Jacques Guilleron
Details
bad arabic curves (32.57 KB, application/vnd.oasis.opendocument.graphics)
2017-12-11 19:56 UTC, Kamil Landa
Details
Example file with text box to convert to curve (8.19 KB, application/vnd.oasis.opendocument.graphics)
2018-02-14 18:29 UTC, Buovjaga
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kamil Landa 2017-12-05 11:34:44 UTC
Description:
After conversion there are extra the small rectangles. 

Steps to Reproduce:
1. In Draw insert Text Box and write arabic text, for example ال or الم.
2. Right click on border of the Text Box and Convert - To Curve.

Actual Results:  
After conversion, in versions 5.4.3.2 & 5.3.7.2 there are extra the small rectangles. In 5.2.7.2 it is OK. 

Expected Results:
Without small rectangles.


Reproducible: Always


User Profile Reset: Yes



Additional Info:
Maybe the unwanted small rectangle is character 1600 (640hexa) for joining some arabic letters. 
I tried it with fonts "Traditional Arabic" and "Arabic Typesetting".


User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.89 Safari/537.36 OPR/49.0.2725.47
Comment 1 Jacques Guilleron 2017-12-11 16:21:59 UTC
Created attachment 138375 [details]
Curves from the text

Hi Kamil,

What I get with
LO 6.0.0.0.beta1 Build ID: 97471ab4eb4db4c487195658631696bb3238656c
Threads CPU : 2; OS : Windows 6.1; UI Render : par défaut; 
Locale : fr-FR (fr_FR); Calc: CL
Where do you see an issue?
Comment 2 Kamil Landa 2017-12-11 19:56:14 UTC
Created attachment 138378 [details]
bad arabic curves
Comment 3 Kamil Landa 2017-12-11 19:58:04 UTC
Jacques, in your example it is OK.
I tried it with some fonts and sizes and for size 18pt it is OK, but for the others -> sometimes OK sometimes bad.


Verze: 5.4.3.2 (x64)
ID sestavení: 92a7159f7e4af62137622921e809f8546db437e5
Vlákna CPU: 2; OS: Windows 6.1; Vykreslování UI: výchozí;
Národní prostředí: cs-CZ (cs_CZ); Calc: CL
Comment 4 Xisco Faulí 2018-01-10 10:22:13 UTC
(In reply to Kamil Landa from comment #3)
> Jacques, in your example it is OK.
> I tried it with some fonts and sizes and for size 18pt it is OK, but for the
> others -> sometimes OK sometimes bad.
> 
> 
> Verze: 5.4.3.2 (x64)
> ID sestavení: 92a7159f7e4af62137622921e809f8546db437e5
> Vlákna CPU: 2; OS: Windows 6.1; Vykreslování UI: výchozí;
> Národní prostředí: cs-CZ (cs_CZ); Calc: CL

Could you please add the steps when it's bad?
Comment 5 Kamil Landa 2018-01-10 16:13:31 UTC
(In reply to Xisco Faulí from comment #4)
> (In reply to Kamil Landa from comment #3)
> > Jacques, in your example it is OK.
> > I tried it with some fonts and sizes and for size 18pt it is OK, but for the
> > others -> sometimes OK sometimes bad.
> > 
> > 
> > Verze: 5.4.3.2 (x64)
> > ID sestavení: 92a7159f7e4af62137622921e809f8546db437e5
> > Vlákna CPU: 2; OS: Windows 6.1; Vykreslování UI: výchozí;
> > Národní prostředí: cs-CZ (cs_CZ); Calc: CL
> 
> Could you please add the steps when it's bad?


It is bad after step 2. Right click on border of the Text Box and Convert - To Curve.
Example is in attachment https://bugs.documentfoundation.org/attachment.cgi?id=138378
Comment 6 Buovjaga 2018-01-30 18:31:32 UTC
(In reply to Kamil Landa from comment #5)
> It is bad after step 2. Right click on border of the Text Box and Convert -
> To Curve.
> Example is in attachment
> https://bugs.documentfoundation.org/attachment.cgi?id=138378

I right clicked several of the boxes and converted to contour, but nothing bad happened.
Maybe try with 6.0 (released tomorrow)

Arch Linux 64-bit
Version: 6.1.0.0.alpha0+
Build ID: 73c757ff71b6bf14206adf13a65213c79928a592
CPU threads: 8; OS: Linux 4.14; UI render: default; VCL: kde4; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group threaded
Built on January 30th 2018
Comment 7 Kamil Landa 2018-02-10 11:53:37 UTC
In 6.0.0.3 it is wrong. I will try 6.0.1.1. 

Verze: 6.0.0.3 (x64)
ID sestavení: 64a0f66915f38c6217de274f0aa8e15618924765
Vlákna CPU: 2; OS: Windows 6.1; Vykreslování UI: výchozí; 
Národní prostředí: cs-CZ (cs_CZ); Calc: CL
Comment 8 Kamil Landa 2018-02-14 15:37:02 UTC
In 6.0.1.1 it is also bad.

Verze: 6.0.1.1 (x64)
ID sestavení: 60bfb1526849283ce2491346ed2aa51c465abfe6
Vlákna CPU: 2; OS: Windows 6.1; Vykreslování UI: výchozí; 
Národní prostředí: cs-CZ (cs_CZ); Calc: CL
Comment 9 Buovjaga 2018-02-14 18:29:28 UTC
Created attachment 139912 [details]
Example file with text box to convert to curve

Ok, I must have been doing something wrong before, because now I could reproduce with Arial font and 40 pts.

It seems to work in 3.6 still. I could try bibisecting later on Windows.

Arch Linux 64-bit
Version: 6.1.0.0.alpha0+
Build ID: 338c1e5ee13ea1435149e04dd61fa19ff476f2a8
CPU threads: 8; OS: Linux 4.15; UI render: default; VCL: kde4; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group
Built on February 14th 2018

Arch Linux 64-bit
Version: 6.0.1.1
Build ID: 6.0.1-1
CPU threads: 8; OS: Linux 4.15; UI render: default; VCL: kde4; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group

Arch Linux 64-bit
Version 3.6.7.2 (Build ID: e183d5b)
Comment 10 Buovjaga 2018-02-17 17:09:21 UTC
Bisect is blaming https://cgit.freedesktop.org/libreoffice/core/commit/?id=5e63617ef934d3602bbc22f0f01c77b024347e60
A change made for bug 103765

Adding Khaled to CC
Comment 11 ⁨خالد حسني⁩ 2018-02-18 18:07:27 UTC
I can’t reproduce this, but round errors causing mysterious kashida insertion is nothing new, it will manifest just randomly. The root issue is how fundamentally broken we do kashida justification, and I’m pretty sure we have an open bug somewhere where this is mentioned.
Comment 12 ⁨خالد حسني⁩ 2018-02-18 18:11:07 UTC
See bug 104921.
Comment 13 Buovjaga 2018-02-18 18:20:33 UTC
Fair enough, let's dupe to that

*** This bug has been marked as a duplicate of bug 104921 ***
Comment 14 Xisco Faulí 2018-06-05 19:43:44 UTC
Adding Cc: to Khaled Hosny
Comment 15 QA Administrators 2019-06-06 02:54:57 UTC Comment hidden (noise)
Comment 16 Kamil Landa 2019-06-06 07:41:56 UTC
Bug is still present, sometime is kashida inserted to the curves and sometime no. 

Verze: 6.2.4.2 (x64)
ID sestavení: 2412653d852ce75f65fbfa83fb7e7b669a126d64
Vlákna CPU: 8; OS: Windows 10.0; Vykreslování UI: výchozí; VCL: win; 
Národní prostředí: cs-CZ (cs_CZ); Jazyk UI: cs-CZ
Calc: threaded

Verze: 6.3.0.0.beta1 (x64)
ID sestavení: a187af327633f5f00363be5131bd21a13e0f1a7b
Vlákna CPU: 8; OS: Windows 10.0; Vykreslování UI: GL; VCL: win; 
Národní prostředí: cs-CZ (cs_CZ); Jazyk UI: cs-CZ
Calc: threaded
Comment 17 QA Administrators 2021-06-06 05:24:32 UTC Comment hidden (noise)
Comment 18 Kamil Landa 2021-06-06 09:58:58 UTC
Bug is still present

Version: 7.1.4.1 (x64) / LibreOffice Community
Build ID: f67b1ddedeb24fca1c5938e7cebfab73d708b35b
CPU threads: 8; OS: Windows 10.0 Build 17763; UI render: Skia/Raster; VCL: win
Locale: cs-CZ (cs_CZ); UI: cs-CZ
Calc: CL

Version: 7.2.0.0.alpha1+ (x64) / LibreOffice Community
Build ID: 3b57ebb445df8a2bc3d916ea79f8af45e20e4e62
CPU threads: 8; OS: Windows 10.0 Build 17763; UI render: Skia/Raster; VCL: win
Locale: cs-CZ (cs_CZ); UI: cs-CZ
Calc: CL
Comment 20 Kamil Landa 2022-08-16 08:13:39 UTC
Vefiried, fixed in 7.5.0.0. 

Version: 7.5.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: c1446da82b999349e1a09fed3420bd1c38d7b38c
CPU threads: 8; OS: Windows 10.0 Build 17763; UI render: Skia/Raster; VCL: win
Locale: cs-CZ (cs_CZ); UI: cs-CZ
Calc: CL