Bug 135810 - FILEOPEN PPTX chart has wrong height
Summary: FILEOPEN PPTX chart has wrong height
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Chart (show other bugs)
Version:
(earliest affected)
7.0.0.2 rc
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks: OOXML-Chart
  Show dependency tree / graph
 
Reported: 2020-08-16 15:47 UTC by Regina Henschel
Modified: 2022-09-30 09:34 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:

Attachments
Chart which gets wrong height (62.38 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2020-08-16 15:47 UTC, Regina Henschel 		Details
The example file in LO 6.4.0 and 7.1 alpha (89.48 KB, image/png)
2020-10-26 13:11 UTC, NISZ LibreOffice Team 		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 Regina Henschel 2020-08-16 15:47:22 UTC 
Created attachment 164356 [details]
Chart which gets wrong height

Open attached document. It has a chart with a small black outer border and a pie-chart with two segments. Behind the chart is an image of it which is a little bit lighter.

Notice, that the chart is compressed from its top and the data label goes outside the border.

(If you activate the chart, the working area is not at the same place as the chart, but shifted. The data labels have different size than in PowerPoint and "best fit" is interpreted differently. But those might be different issues.)

I have used Version: 7.0.0.2 (x64)
Build ID: c01aa64b6c3d89ebe5fe69c28c7adb24eb85249c
CPU threads: 8; OS: Windows 10.0 Build 18362; UI render: Skia/Raster; VCL: win
Locale: de-DE (en_US); UI: en-US
Calc: CL

and
Version: 7.1.0.0.alpha0+ (x64)
Build ID: edd916b47e01ea53f0989bdbb6a703e081e25aea
CPU threads: 8; OS: Windows 10.0 Build 18362; UI render: Skia/Raster; VCL: win
Locale: de-DE (en_US); UI: en-US
Calc: CL
Comment 1 Xisco Faulí 2020-09-07 09:09:47 UTC 
Regression introduced by:

https://cgit.freedesktop.org/libreoffice/core/commit/?id=73477348e30c6931a537cba5557c250183fbeb9b

author	Balazs Varga <balazs.varga991@gmail.com>	2020-05-26 14:13:00 +0200
committer	László Németh <nemeth@numbertext.org>	2020-06-02 14:07:08 +0200
commit 73477348e30c6931a537cba5557c250183fbeb9b (patch)
tree 47d9e863d4453de74bcb0d309eaaa18a41da4a04
parent a9b47360c2af2514b1064fb179f9fb9f0b8ad225 (diff)
tdf#133376 Chart view: improve BestFit position of data labels

Bisected with: bibisect-linux64-7.1
 
Adding Cc: to Balazs Varga
Comment 2 NISZ LibreOffice Team 2020-10-26 13:11:41 UTC 
Created attachment 166736 [details]
The example file in LO 6.4.0 and 7.1 alpha

How it looks in:

Verzió: 6.4.0.3 (x86)
Build az.: b0a288ab3d2d4774cb44b62f04d5d28733ac6df8
CPU szálak: 4; OS: Windows 6.3 Build 9600; Felületmegjelenítés: alapértelmezett; VCL: win; 
Területi beállítások: en-US (hu_HU); Felület nyelve: hu-HU
Calc: CL

and
Version: 7.1.0.0.alpha1+ (x64)
Build ID: b61bf7c7cfcf97a5ade6d130873af146670bc2ee
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win
Locale: hu-HU (hu_HU); UI: en-US
Calc: CL

The chart shape became a bit more ellipsoid.

Another source of difference is that in MSO the data labels (by default) have a bit of inner margin which is not supported by LO.
So the label to be positioned gets a bit smaller, making it fit in tighter places too.