Bug 118018 - Garbled text in chart
Summary: Garbled text in chart
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.1.0.0.alpha1+
Hardware: All Windows (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: CommonSalLayout-refactoring-regressions
  Show dependency tree / graph
 
Reported: 2018-06-05 18:22 UTC by Xisco Faulí
Modified: 2018-07-10 14:02 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
sample document (45.64 KB, application/vnd.oasis.opendocument.text)
2018-06-05 18:22 UTC, Xisco Faulí 		Details
How it looks in LibreOffice 6.2 master (15.02 KB, image/png)
2018-06-05 18:22 UTC, Xisco Faulí 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Xisco Faulí 2018-06-05 18:22:30 UTC 
Created attachment 142545 [details]
sample document

Steps to reproduce:
1. Open attached document

-> See the red text in the chart. See attached screenshot

Reproduced in

Version: 6.2.0.0.alpha0+
Build ID: 48b49937fed5e50d299a94063eb325799ff672e9
CPU threads: 1; OS: Windows 6.1; UI render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2018-06-05_01:47:06
Locale: es-ES (es_ES); Calc: group threaded
Comment 1 Xisco Faulí 2018-06-05 18:22:59 UTC 
Created attachment 142546 [details]
How it looks in LibreOffice 6.2 master
Comment 2 Xisco Faulí 2018-06-05 18:25:06 UTC 
Regression introduced by:

author	Jan-Marek Glogowski <glogow@fbihome.de>	2017-12-26 15:14:31 +0000
committer	Khaled Hosny <khaledhosny@eglug.org>	2018-05-08 00:55:27 +0200
commit bdccb7e9991d83029eb2f2f11327b54534a00db8 (patch)
tree c32e95c49849647dc72c1071f375f3d2b67d8d7a
parent 9615e45d2e2bac79c252a018846e4f20012cfa34 (diff)
Refactor CommonSalLayout font handling
Moves all platform specific code from CommonSalLayout into the
platform specific plugins. This way the vcl library won't depend
on the Qt5 libraries and the Qt5Font header can be moved into the
qt5 VCL plugin.

While at it, switch the CommonSalLayouts font reference from the
FontSelectPattern to the LogicalFontInstance and also add the
harfbuzz font handling to the instance.

Bisected with: bibisect-win32-6.1

Adding Cc: to Jan-Marek Glogowski
Comment 3 Xisco Faulí 2018-07-10 13:54:49 UTC 
Verified in

Version: 6.2.0.0.alpha0+
Build ID: c290f692dd28094d41dff686f3faa1c4e14b556e
CPU threads: 16; OS: Windows 6.3; UI render: default; 
Locale: en-GB (en_GB); Calc: group threaded

@Jan-Marek, thanks for fixing this! Should it be closed as RESOLVED FIXED?