Created attachment 182523 [details] wrapOptimal.xls: optimal height is enabled and functional This document has optimal height enabled for row 1. It works too. (Can be confirmed by adding extra text and watching the height increase.) However, when saving to ODS format, the first save lists autostyle ro1 as use-optimal-row-height = false. (A second save immediately afterwards correctly sets = true.) This is only seen on an immediate round-trip. If you make any change to row 1, then it will get the autostyle correctly. I thought perhaps that running ScDocRowHeightUpdater::updateAll() might avoid this problem, but it doesn't. For some reason, the same thing is not true for XLSX import. True in master, and in 5.1, and in 3.6. Assuming inherited from OOo Steps to reproduce 1.) open wrapOptimal.xls 2.) save as ODT format. 3.) reload ODT - add content to A1 and notice that the cell height doesn't grow.
*** This bug has been marked as a duplicate of bug 130383 ***
I set the bug 130383 to fixed, but this one can still be reproduced. Even worse than reported: a second save does not fix the issue. Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community Build ID: 25276df12abd9d002f7f899900434617b256f745 CPU threads: 15; OS: Windows 10.0 Build 19045; UI render: Skia/Raster; VCL: win Locale: de-DE (hu_HU); UI: de-DE Calc: threaded
(In reply to Gabor Kelemen (allotropia) from comment #2) > Even worse than reported: a second save does not fix the issue. Second save still works for me in today's 24.8. Let me clarify my steps. 1.) open wrapOptimal.xls 2a.) save as wrapOptimalRT1.ods 2b.) (and then immediately) save as wrapOptimalRT2.ods 3a.) reload RT1 - new content doesn't expand the cell height. 3b.) reload RT2 - new content does expand the cell height
commit e623ee66dfde0d5a2705e222aaa7ce580e409506 Author: Noel Power on Tue May 22 10:08:26 2012 +0100 make row heights ( incl default row height ) from excel are saved absolute QUOTE: "Currently default row heights imported from Excel are not adjusted Optimally ( AdjustRowHeight() is no longer called ) However to ensure correct round tripping the CR_MANUALSIZE flag *is* still set, this results in the style:use-optimal-row-height="true" being set for default row height(s) when saving as odf. So, for example absolute positions which are calculated on importing the Excel document ( where the row height is *not* optimally adjusted ) are still saved with the 'style:use-optimal-row-height' set. When the row heights are read back into calc the row sizes are changed any absolutely positioned objects are now out of place. This patch ensures *all* row heights are from an imported excel document are saved with style:use-optimal-row-height="false" " This has aXMLScFromXLSRowStylesProperties force optimal-row-height to be false with XML_TYPE_BOOL_FALSE, since it isAlienExcelDoc. Since this is very intentional, and the explanation is complete gobbledygook, I'm walking away...
Dear Justin L, To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year. There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present. If you have time, please do the following: Test to see if the bug is still present with the latest version of LibreOffice from https://www.libreoffice.org/download/ If the bug is present, please leave a comment that includes the information from Help - About LibreOffice. If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a comment that includes the information from Help - About LibreOffice. Please DO NOT Update the version field Reply via email (please reply directly on the bug tracker) Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not appropriate in this case) If you want to do more to help you can test to see if your issue is a REGRESSION. To do so: 1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) from https://downloadarchive.documentfoundation.org/libreoffice/old/ 2. Test your bug 3. Leave a comment with your results. 4a. If the bug was present with 3.3 - set version to 'inherited from OOo'; 4b. If the bug was not present in 3.3 - add 'regression' to keyword Feel free to come ask questions or to say hello in our QA chat: https://web.libera.chat/?settings=#libreoffice-qa Thank you for helping us make LibreOffice even better for everyone! Warm Regards, QA Team MassPing-UntouchedBug