Bug 150176 - FILEOPEN PPTX: extraneous borders around chart objects
Summary: FILEOPEN PPTX: extraneous borders around chart objects
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.3.0.4 release
Hardware: All All
: medium normal
Assignee: Balázs Varga (allotropia)
URL:
Whiteboard: target:7.5.0 target:7.4.1
Keywords: bibisected, bisected, regression
Depends on:
Blocks: OOXML-Chart
  Show dependency treegraph
 
Reported: 2022-07-27 22:21 UTC by Gerald Pfeifer
Modified: 2022-08-12 21:30 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Two sample slides (PPTX) (66.34 KB, application/vnd.ms-powerpoint)
2022-07-27 22:21 UTC, Gerald Pfeifer
Details
Visual comparison Office 365 (left) vs LibreOffice (right) (91.15 KB, image/png)
2022-07-27 22:23 UTC, Gerald Pfeifer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gerald Pfeifer 2022-07-27 22:21:38 UTC
Created attachment 181458 [details]
Two sample slides (PPTX)

Comparing the sample document (both slides) between Powerpoint and
Impress, the latter shows extraneous lines around the chart objects.


Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 5df1bb4b1b222be00d25097660c4ee33542896ea
CPU threads: 8; OS: Linux 5.18; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US

...and all the way back to

Version: 7.0.7.0.0+
Build ID: 54e9dd41dc9dd45af12c9346199f601ea4a5994d
CPU threads: 8; OS: Linux 5.18; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:libreoffice-7-0, Time: 2021-05-07_08:22:18
Comment 1 Gerald Pfeifer 2022-07-27 22:23:54 UTC
Created attachment 181459 [details]
Visual comparison Office 365 (left) vs LibreOffice (right)
Comment 2 Gerald Pfeifer 2022-07-27 22:25:01 UTC
This looks similar to bug #147617, alas that one was a regression, has
been fixed in the meantime, and only manifested in presentation mode
whereas this issue shows both in edit mode and presentation mode (and
dates back "forever").
Comment 3 Aron Budea 2022-07-28 11:16:28 UTC
Confirmed using LO 7.5.0.0.alpha0+ (c4f7043c593823b8c5605e779371ff430659eb20) / Ubuntu.

This is a regression, and could be bibisected to the following commit using the 6.3 linux repo. Adding CC: to Balazs Varga.

https://cgit.freedesktop.org/libreoffice/core/commit/?id=0dac43445b35da7e554dd4d7574c59611a0d0be1
author		Balazs Varga <balazs.varga991@gmail.com>	2019-03-16 18:05:01 +0100
committer	László Németh <nemeth@numbertext.org>	2019-03-18 16:38:55 +0100

tdf#81437 XLSX import: fix missing chart border
Comment 4 Commit Notification 2022-08-09 09:36:45 UTC
Balazs Varga committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/071a36e042c76286fedb38f479dac79f29b661f9

tdf#150176 pptx chart import: fix automatic border style

It will be available in 7.5.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 5 Commit Notification 2022-08-09 13:37:37 UTC
Balazs Varga committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

https://git.libreoffice.org/core/commit/57bfc05e9f0b2517e0922d1bb94902e49f6c6faa

tdf#150176 pptx chart import: fix automatic border style

It will be available in 7.4.1.

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

Affected users are encouraged to test the fix and report feedback.
Comment 6 Gerald Pfeifer 2022-08-10 08:44:53 UTC
Thank you, Balazs!

Happy to confirm as fixed with today's dev build:

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 4e2ce2a460458f17ee4360c45a2da2fc4b4d753e
CPU threads: 8; OS: Linux 5.18; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 7 Commit Notification 2022-08-11 06:30:28 UTC
Balazs Varga committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/f810f05e2b50068f6d14be152eb4c2ffbc1c4e5e

Related: tdf#150176 pptx chart import: fix automatic border style

It will be available in 7.5.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 8 Commit Notification 2022-08-12 21:30:40 UTC
Balazs Varga committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

https://git.libreoffice.org/core/commit/265ba86ef25044d9ed0ebadbb63da6253bcb5964

Related: tdf#150176 pptx chart import: fix automatic border style

It will be available in 7.4.1.

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

Affected users are encouraged to test the fix and report feedback.