Bug 124953

Summary:	FILESAVE, FORMATTING: xlsx file with conditional formatting created in Excel roundtripped by Calc does not look the same in Excel again
Product:	LibreOffice	Reporter:	Dennis Francis <dennisfrancis.in>
Component:	Calc	Assignee:	Dennis Francis <dennisfrancis.in>
Status:	RESOLVED FIXED
Severity:	normal	Keywords:	filter:xlsx
Priority:	medium
Version:	6.3.0.0.alpha0+
Hardware:	All
OS:	All
Whiteboard:	target:6.3.0
Crash report or crash signature:		Regression By:
Bug Depends on:
Bug Blocks:	113782
Attachments:	xlsx file created in Excel from scratch.

Description Dennis Francis 2019-04-25 11:42:32 UTC

Description:
The attached file is created from scratch in Excel. Opening this in Calc and saving it as another xlsx file and opening this new xlsx file in Excel shows misplaced conditional formatting colors.

Steps to Reproduce:
1. Open the attached xlsx file in Calc
2. Notice that B5 and G2 have green background via conditional formatting.
3. Save this as a new xlsx file.
4. Open this new file in Excel.
5. Observe the background color of cells B5 and G2. Also observe background color of B4.

Actual Results:
Cells B5 and G2 do not have green background anymore, where as B4 is now colored green !

Expected Results:
Cells B5 and G2 must have retained the green background and B4 should not have any background color for the round-tripped file when viewed in Excel.


Reproducible: Always


User Profile Reset: No


OpenGL enabled: Yes

Additional Info:
Version: 6.3.0.0.alpha0+
Build ID: 74a7b890524af4a52e711d8f0793ce844ded0948
CPU threads: 8; OS: Linux 5.0; UI render: default; VCL: gtk3; 
Locale: en-US (en_US.utf8); UI-Language: en-US
Calc: threaded

Comment 1 Dennis Francis 2019-04-25 11:43:42 UTC

Created attachment 151006 [details]
xlsx file created in Excel from scratch.

Comment 2 Commit Notification 2019-05-04 08:23:25 UTC

Dennis Francis committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/+/ca40d4ce9b8134fd8bfc90e9e8289b620163475b%5E%21

tdf#124953: Use rangelist's combined range top-left address...

It will be available in 6.3.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.