Bug 123613 - FILESAVE XLSX Rotated shapes change their X/Y position on export
Summary: FILESAVE XLSX Rotated shapes change their X/Y position on export
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium normal
Assignee: Szabolcs Toth
URL:
Whiteboard: target:7.2.0 target:7.1.0.0.beta2
Keywords: filter:xlsx
: 103050 (view as bug list)
Depends on:
Blocks: XLSX-Shapes
  Show dependency treegraph
 
Reported: 2019-02-21 09:36 UTC by NISZ LibreOffice Team
Modified: 2021-01-18 09:56 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
The original file (74.00 KB, application/vnd.ms-excel)
2019-02-21 09:36 UTC, NISZ LibreOffice Team
Details
Screenshot of the original and exported document side by side in Writer (348.44 KB, image/png)
2019-02-21 09:37 UTC, NISZ LibreOffice Team
Details
Screenshot of the original file and its XLSX version in current Calc (139.27 KB, image/png)
2020-08-28 10:41 UTC, NISZ LibreOffice Team
Details
Better example file with shapes rotated in all directions (9.96 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2020-08-28 10:51 UTC, NISZ LibreOffice Team
Details
The example file saved with Calc master (6.54 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2020-08-28 10:52 UTC, NISZ LibreOffice Team
Details
The better example file and its XLSX-saved version in Calc (154.76 KB, image/png)
2020-08-28 10:54 UTC, NISZ LibreOffice Team
Details
The better example file and its XLSX-saved version in Excel (167.03 KB, image/png)
2020-08-28 10:55 UTC, NISZ LibreOffice Team
Details

Note You need to log in before you can comment on or make changes to this bug.
Description NISZ LibreOffice Team 2019-02-21 09:36:21 UTC
Description:
When the attached file is saved in LO as xlsx the arrows move out of their original place and lose their text. Also if you open the exported file in MSO all of the arrows gets resized.

Steps to Reproduce:
    1. Open the attached original xlsx in LO.
    2. Save it as xlsx.
    3. Reload.
    4. Notice the changes.

Actual Results:
The arrows are not in their original place and they are missing their white text.

Expected Results:
The arrows should stay in their original place and keep their text.


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 NISZ LibreOffice Team 2019-02-21 09:36:50 UTC
Created attachment 149456 [details]
The original file
Comment 2 NISZ LibreOffice Team 2019-02-21 09:37:17 UTC
Created attachment 149457 [details]
Screenshot of the original and exported document side by side in Writer
Comment 3 Durgapriyanka 2019-02-22 18:53:06 UTC
Thank you for reporting the bug. I can confirm the bug present in

Version: 6.3.0.0.alpha0+
Build ID: b6b28931435e44aca92b8c0e1659f701e3ed1a87
CPU threads: 2; OS: Windows 6.1; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-01-30_06:57:04
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded
Comment 4 Buovjaga 2019-04-07 19:24:52 UTC
In old versions the transformations affected all the arrows so it was much worse.
Comment 5 NISZ LibreOffice Team 2020-08-28 10:41:14 UTC
Created attachment 164793 [details]
Screenshot of the original file and its XLSX version in current Calc

After bug #135828 was fixed the export looks better: rotated arrows thickness/narrowness is the ~same as the original.

Remaining problem is that rotated shapes change their X/Y position on export.
Comment 6 NISZ LibreOffice Team 2020-08-28 10:51:39 UTC
Created attachment 164794 [details]
Better example file with shapes rotated in all directions
Comment 7 NISZ LibreOffice Team 2020-08-28 10:52:37 UTC
Created attachment 164795 [details]
The example file saved with Calc master

Version: 7.1.0.0.alpha0+ (x86)
Build ID: 6640d7f405d2970ba2825a9455926cc803284d01
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: hu-HU (hu_HU); UI: en-US
Calc: CL
Comment 8 NISZ LibreOffice Team 2020-08-28 10:54:38 UTC
Created attachment 164796 [details]
The better example file and its XLSX-saved version in Calc

Shapes had to be colored manually in the reopened version because of bug #124013
Comment 9 NISZ LibreOffice Team 2020-08-28 10:55:34 UTC
Created attachment 164797 [details]
The better example file and its XLSX-saved version in Excel

Positioning is just as wrong in Excel as in Calc.
Comment 10 Commit Notification 2020-12-11 09:10:47 UTC
Szabolcs Toth committed a patch related to this issue.
It has been pushed to "master":

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

tdf#123613 XLSX export: fix position of rotated shapes

It will be available in 7.2.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 11 Commit Notification 2020-12-11 16:48:47 UTC
Szabolcs Toth committed a patch related to this issue.
It has been pushed to "libreoffice-7-1":

https://git.libreoffice.org/core/commit/8f7c26f166df33589f3522b43dbea4169a9ffd1e

tdf#123613 XLSX export: fix position of rotated shapes

It will be available in 7.1.0.0.beta2.

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 12 NISZ LibreOffice Team 2021-01-11 10:28:15 UTC
Verified in:

Version: 7.2.0.0.alpha0+ (x64)
Build ID: 96bafa464ebdbce3ef04bec9beae5e745bb37794
CPU threads: 4; OS: Windows 10.0 Build 17134; UI render: Skia/Raster; VCL: win
Locale: hu-HU (hu_HU); UI: hu-HU
Calc: threaded

Thanks for fixing!
Comment 13 Szabolcs Toth 2021-01-18 09:56:34 UTC
*** Bug 103050 has been marked as a duplicate of this bug. ***