Bug 84647

Summary: FILEOPEN: text flow parameters incorrectly set in .XLSX causing some labels to disappear
Product: LibreOffice Reporter: Joey Reid <joeyisdamanya>
Component: SpreadsheetAssignee: Sushil Shinde <sushilshinde>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: medium CC: barta, erack
Version: Inherited From OOo   
Hardware: Other   
OS: All   
See Also: https://bugs.freedesktop.org/show_bug.cgi?id=85491
Whiteboard: target:4.4.0
i915 platform: i915 features:
Attachments: Problem with Overlap and Break import settings with XLS on Left and XLSX on right
Sample XLSX chart with missing labels
Same file but saved as XLS correctly imported

Description Joey Reid 2014-10-04 03:20:34 UTC
Created attachment 107295 [details]
Problem with Overlap and Break import settings with XLS on Left and XLSX on right

When Calc imports Excel XLS charts, it correctly sets:
 
Text Flow:Overlap-enabled
Text FLow:Break-enabled

But for XLSX charts it sets

Text Flow:Overlap-DISABLED
Text FLow:Break-DISABLED

This causes visual data loss, as the labels disappear. In the screenshot, you can see every other label is missing in the XLSX file.
Comment 1 Joey Reid 2014-10-04 03:21:36 UTC
Created attachment 107296 [details]
Sample XLSX chart with missing labels
Comment 2 Joey Reid 2014-10-04 03:22:17 UTC
Created attachment 107297 [details]
Same file but saved as XLS correctly imported
Comment 3 tommy27 2014-10-05 07:52:28 UTC
confirmed under Win7x64 using 4.3.2.2 and 4.4.0.0.alpha0+
Build ID: 268b9c10c9ff27c74678ace99762f28d58d33012
TinderBox: Win-x86@42, Branch:master, Time: 2014-10-02_23:35:24

it seems that 50% of the labels in the x axis are not shown

same issue even with older LibO release (tried with 3.4.3) and in AOO 4.1.0 as well so the bug is likely to be inherited from OOo
Comment 4 Adolfo Jayme 2014-10-11 17:11:08 UTC
Sushil Shinde committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=4027bc476a1df9df1924447795ddc2d8b7744e20

fdo#84647 : Fixed default value for TickLableSkip value.



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 Joey Reid 2014-10-26 19:08:38 UTC
Steps to reproduce:
1. Create a chart in Excel with long label names (2 or more lines)
2. Save as .XLS and .XLSX
3. Open in Calc

When Calc imports Excel XLS charts, it correctly sets:
Text FLow:Break-enabled

But for XLSX charts it sets
Text FLow:Break-DISABLED

This causes as the labels in XLSX to overlap and makes them unreadable.
Comment 6 tommy27 2014-10-26 19:38:38 UTC
@Joey Reid  
have you tested that with recent 4.0.x daily builds? those are the only LibO versions featuring that bugfix
Comment 7 Joey Reid 2014-10-26 19:49:00 UTC
Tommy,
Yes, Sushil fixed the Text Flow:Overlap default. I verified that's it's working now with the latest 4.4 builds. Instead of messing up this report, I just now created Bug 85491 for the Text FLow:Break default.
Comment 8 Luke 2014-12-27 03:18:57 UTC
Sushil,
Should this bug report be reopened? In recent builds Text Flow:Overlap is not being set any more.
Comment 9 Sushil Shinde 2014-12-27 05:37:38 UTC
Hi Luke,

This issue was fixed in two parts,
1. https://gerrit.libreoffice.org/#/c/11901/
2. https://gerrit.libreoffice.org/#/c/12213/
Second patch is not yet merged to master. If possible, Please apply these patches and verify.
If this issue still reproducible then reopen it. I will look into it.

Thanks.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.