Bug 114913 - FILEOPEN: PPTX: Bullets size is too small
Summary: FILEOPEN: PPTX: Bullets size is too small
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.1.0.0.alpha0+
Hardware: All All
: medium normal
Assignee: Paul Trojahn
URL:
Whiteboard: target:6.1.0
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2018-01-08 15:06 UTC by Xisco Faulí
Modified: 2018-01-25 14:51 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
Sample file (40.21 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2018-01-08 15:06 UTC, Xisco Faulí 		Details
comparison MSO 2010 and LibreOffice 6.1 (73.03 KB, image/png)
2018-01-08 15:07 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-01-08 15:06:57 UTC 
Created attachment 138972 [details]
Sample file

Steps to reproduce:
1. Open attached document.

Check the bullets size ( See the image attached )

Reproduced in

Version: 6.1.0.0.alpha0+
Build ID: 0ef0740298b45379bbf8d00d50beffee7a2f812a
CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded
Comment 1 Xisco Faulí 2018-01-08 15:07:53 UTC 
Created attachment 138973 [details]
comparison MSO 2010 and LibreOffice 6.1
Comment 2 Xisco Faulí 2018-01-08 15:09:31 UTC 
Regression introduced by:

author	Paul Trojahn <paul.trojahn@gmail.com>	2017-11-13 18:27:22 +0100
committer	Tamás Zolnai <tamas.zolnai@collabora.com>	2017-12-18 15:46:40 +0100
commit 3b4f14f4da046dab25362114db737968634cd720 (patch)
tree 1d11378090d4d5d4e8b8a6cfa605c28b7a33830b
parent 61b1d631331551b43bc7d619be33bfbfeff7cad6 (diff)
tdf#90626 PPTX: Fix image bullet size
This solution comes with a few limitations. The bullet isn't centered in
Powerpoint and has to be between 25% and 400% of the size of the first
character in the paragraph. This isn't optimal, but should produce
acceptable results under most circumstances.

Bisected with: bibisect-linux64-6.1

Adding Cc: to Paul Trojahn

Before the commit, the bullets look different than on MSO but now they look completely different.
Comment 3 Xisco Faulí 2018-01-08 15:13:09 UTC 
[Bug found by office-interoperability-tools]
Comment 4 Commit Notification 2018-01-24 16:22:57 UTC 
Paul Trojahn committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=1fe4f041aeef2fcabd90bc07ab6a5ee5aba1dd59

tdf#114913 Use master if char size wasn't set

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 5 Xisco Faulí 2018-01-25 14:51:31 UTC 
Verified in

Version: 6.1.0.0.alpha0+
Build ID: 5cb225b0b3dbb55fd44899b1fb16bdcfeb866849
CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded

Thanks for the fix Paul !!