Description: The line spacing in Calc has changed between LibO 5.3.0.3 and previous releases (tested with 5.1.4.2). Steps to Reproduce: From new to old: 1. Open Calc with LibO 5.3.0.3. 2. Write text with line breaks in a cell. 3. Save it and open the file in an older LibO version. 4. The line spacing is different (bigger) and therefore the first line of the text is partially hidden. Other direction from old to new: 1. Open Calc with LibO 5.1.4.2 or another old version. 2. Write text with line breaks in a cell. 3. Save it and open the file in LibO 5.3.0.3. 4. The line spacing is different (smaller) and therefore the first line of the text has more padding to the upper border of the cell. Direct comparison: 1. Write text with line breaks in a cell in both LibO 5.3.0.3 and an older version. 2. Compare the documents side by side. 3. You will see that the line spacing in LibO 5.3.0.3 is much smaller than in the older version. Actual Results: Line spacing differs between versions. LibO 5.3.0.3 has a much smaller spacing than older versions. The text looks crowded together and not so good readable in LibO 5.3.0.3. Expected Results: There should be no difference in line spacing if the same font on the same OS version is used. Maybe in the future a line spacing option should be available for the users to change it by themselves. Reproducible: Always User Profile Reset: Yes. The test in version 5.3.0.3 was made with a new profile. Additional Info: [Information automatically included from LibreOffice and me] Locale: de Module: SpreadsheetDocument OS version: Windows 6.2 UI render: Standard Layout engine: neu (new) [Information guessed from browser] OS is 64bit: yes Builds ID: LibreOffice 5.3.0.3 Builds ID: LibreOffice 5.3.0.3 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:47.0) Gecko/20100101 Firefox/47.0
Created attachment 131016 [details] Line spacing in 5.1.4.2 and 5.3.0.3
Created attachment 131017 [details] Test file made with 5.3.0.3
Created attachment 131018 [details] Test file made with 5.1.4.2
Nicely prepared bug report. I don't see why this would be a bug, maybe some change from https://wiki.documentfoundation.org/ReleaseNotes/5.3. I'm not an expert, and I won't change the status.
Bug 55469 is mentioned at the 5.3 release notes wiki page. But I don't see why changes in Math visualization should have effected spreadsheet visualization. Is this possible? I will add Khaled Hosny to CC list, maybe he knows more about this. Timur: I think this is a bug because the new behavior disturb the continuity of OpenDocument file presentation. Users are rightly assuming that their files looks the same in different LibO versions (especially when NO file format changes were made as in ODF 1.0 > ODF 1.2 or DOC > ODF). Apart from that the new line spacing looks ugly (too small/crowded).
Regression introduced by: author Khaled Hosny <khaledhosny@eglug.org> 2016-11-09 13:22:43 (GMT) committer Khaled Hosny <khaledhosny@eglug.org> 2016-11-22 15:32:11 (GMT) commit 34d7602954d4483b3bc9db700e7df2c15348947a (patch) tree 8dcfb93fc29815fd89481a7840d64d6c187534db parent c855aec445628f96d3d32cfde6efd4e51e4489c9 (diff) tdf#55469 Consistent line spacing across platforms
We were already using different spacing on Linux and Mac, so at least we are now consistent across platforms. I don’t consider this a regression, it is by design to fix how line spacing is calculated once and for all, which fixes issues like bug 55469. This is not a bug AFAIC, however if we think strongly about preserving backward compatibility here (including the bugs, of course) then I think a flag needs to be set somewhere for old documents and propagated to VCL that signals calculating line spacing the old way.
Khaled, thanks for your work on making spacing consistent across platforms. Overall, that's good and makes things easier in the future. Based on your statement I would rephrase my bug report: The line spacing is too small now (it seems that this was true in the past on Linux/Mac). Multi-line content looks crowded together and not so good readable. Other programs use line spacing that facilitates readability in a better way.
Created attachment 131075 [details] Second test: file made with LibO 5.3.0.3
Created attachment 131076 [details] Second test: file made with Excel 2010
(In reply to Thomas Lendo from comment #9) > The line spacing is too small now (it seems that this was true in the past > on Linux/Mac). > .. > Created attachment 131075 [details] Looks not bad to me. But there must not be such a difference to Excel, depending on how you create the examples. @Stuart What's your opinion?
*** Bug 106111 has been marked as a duplicate of this bug. ***
(In reply to Khaled Hosny from comment #7) > We were already using different spacing on Linux and Mac, so at least we are > now consistent across platforms. > > I don’t consider this a regression, it is by design to fix how line spacing > is calculated once and for all, which fixes issues like bug 55469. > > This is not a bug AFAIC, however if we think strongly about preserving > backward compatibility here (including the bugs, of course) then I think a > flag needs to be set somewhere for old documents and propagated to VCL that > signals calculating line spacing the old way. After bug 106111, which is introduced by 34d7602954d4483b3bc9db700e7df2c15348947a, this can be considered a regression in my humble opinion...
@Xisco: Thank you very much for the successful bibisect! This patch makes so much trouble and must be revoked asap with effect for the next bugfix release. Is it already revoked from the branch?
Are comment 12 to 15 obsolete for this bug? I change the bug title from "VIEWING: line spacing different between program versions" to "VIEWING: line spacing is too small compared to older versions and other programs".
Seems bug 107249 is also linked to the same commit by Khaled, so lets link these together. (In reply to Khaled Hosny from comment #7) > I don’t consider this a regression, it is by design to fix how line spacing > is calculated once and for all, which fixes issues like bug 55469. I would consider there is a bug in the calculation if it is cropping letters from labels as mentioned in bug 107249, or do you consider this a bug in the fonts (Noto Sans, Ubuntu) themselves?
ESC decided to keep the patch with the new calculation. So this ticket is a WONTFIX and we can talk about new defaults.
More details: Changing the zoom affects the line spacing - quite significantly between 100% and 110%. This zoom effect is plainly evident in the multi-line entry of the test file provided with this bug (Comment 2) - look at the vertical fit of the row 1 text within that row as the zoom is changed. This I believe is the same issue as described in my Bug 108710 for Writer. Word and Excel do not exhibit this problem when viewing these files.
*** Bug 108710 has been marked as a duplicate of this bug. ***
The problem has not been resolved. Khaled has done good work to "fix how line spacing is calculated once and for all". I am sure he has achieved that for the issues he was dealing with. However, his work appears to only address the rendering of text onto a page. The problem many of us are seeing, is when the text is being rendered in abstraction - where the text is NOT being shown in the context of a physical page. In that situation there seems to be a completely different calculation being used for line spacing, and it is affected by the zoom percentage. The calculation differences become particularly noticeable when there are associated elements that are assigned a physical size e.g. row height. THIS line-spacing calculation problem has existed for a long time, but has become worse and much more noticeable from version 5.3
(In reply to Dale from comment #21) > The problem has not been resolved. The OP closed this ticket as superseded by bug 108891. Please comment there.