Bug 88476 - Justify changes cell height
Summary: Justify changes cell height
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.3.5.2 release
Hardware: x86-64 (AMD64) All
: medium major
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-15 21:35 UTC by Robert
Modified: 2020-05-23 11:05 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Simple spreadsheet exhibiting the problem (62.61 KB, application/vnd.oasis.opendocument.spreadsheet)
2015-01-15 21:35 UTC, Robert
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Robert 2015-01-15 21:35:11 UTC
Created attachment 112320 [details]
Simple spreadsheet exhibiting the problem

See the attached file.

Select any cell and click on "Justify". The height of the row containing the cell will visibly change.
Comment 1 Robinson Tryon (qubit) 2015-01-15 22:55:30 UTC
TESTING with LO 4.4.0.2 + Ubuntu 14.04

(In reply to Robert from comment #0)

REPRO Steps:
> Open attachment 112320 [details] 
>
> Select any cell 

E8 was already selected, so I went with that.

> and click on "Justify". 

I had to enable a toolbar to see the Justify button:
View -> Toolbars -> Text Formatting

> The height of the row containing the
> cell will visibly change.

CONFIRMED: row height appears to increase slightly.

Status -> NEW

Robert: Is this a regression? Did previous versions justify without increasing the line height?
Comment 2 Cor Nouws 2015-01-17 10:28:49 UTC
[there are more moments that row hight changes, especially in xls files is my idea. But - apologies - apart from one bug I filed, never bothered to create reproducible situations to file issues ]
Comment 3 Robert 2015-01-19 00:41:14 UTC
(In reply to Robinson Tryon (qubit) from comment #1)
> TESTING with LO 4.4.0.2 + Ubuntu 14.04
> 
> (In reply to Robert from comment #0)
> 
> REPRO Steps:
> > Open attachment 112320 [details] 
> >
> > Select any cell 
> 
> E8 was already selected, so I went with that.
> 
> > and click on "Justify". 
> 
> I had to enable a toolbar to see the Justify button:
> View -> Toolbars -> Text Formatting
> 
> > The height of the row containing the
> > cell will visibly change.
> 
> CONFIRMED: row height appears to increase slightly.
> 
> Status -> NEW
> 
> Robert: Is this a regression? Did previous versions justify without
> increasing the line height?

I do not know if this is a regression or not, I never actually intended to click Justify, but Fat-Mouse'd...
Comment 4 andréb 2015-02-26 18:23:53 UTC
It is due the added minimal-row-height in libo 4.2
Any change in cells in the line will increase the height to 4,5mm if it is less.

In my view this is a bug.  Such a "feature" serves no useful purpose, and reduces the lines available for display by 20% if one had 8px text before (which had an automatic line height of 3,6mm).

A 20% reduction of display content is a significant loss, so I would classify this as "major", since the regression is easily fixed in source code.
Comment 5 tommy27 2016-04-16 07:28:27 UTC Comment hidden (obsolete)
Comment 6 robert 2016-04-16 14:55:42 UTC
W7-64 Pro (completely patched - minus M$ telemetry crap) on Intel i7-4710MQ, 

Libreoffice:

Version: 5.1.2.2 (x64)
Build ID: d3bf12ecb743fc0d20e0be0c58ca359301eb705f
CPU Threads: 8; OS Version: Windows 6.1; UI Render: GL; 
Locale: en-GB (en_GB)

Bug still present!
Comment 7 QA Administrators 2017-05-22 13:26:31 UTC Comment hidden (obsolete)
Comment 8 robert 2017-05-22 15:04:55 UTC
Version: 5.3.3.2 (x64)
Build ID: 3d9a8b4b4e538a85e0782bd6c2d430bafe583448
CPU Threads: 8; OS Version: Windows 6.1; UI Render: default; Layout Engine: new; 
Locale: en-GB (en_GB); Calc: group

Bug still present!
Comment 9 QA Administrators 2018-05-23 02:35:44 UTC Comment hidden (obsolete)
Comment 10 QA Administrators 2020-05-23 03:43:39 UTC Comment hidden (obsolete)
Comment 11 robert 2020-05-23 11:05:55 UTC
Version: 6.4.4.2 (x64)
Build ID: 3d775be2011f3886db32dfd395a6a6d1ca2630ff
CPU threads: 8; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: default; VCL: win; 
Locale: en-GB (en_GB); UI-Language: en-US
Calc: threaded

Bug is still present!