https://bugs.freedesktop.org/show_bug.cgi?id=81568

--- Comment #5 from Jorendc <[email protected]> ---
Hi Miklos,

When I place the insertion of PROP_HEIGHT outside your introduced if-statement
of commit 970160f78ef6cc7abacfa252daa8451e1f0117bb , this zero-col-height looks
fixed. But counterside is that your unittest of that commit fails (actual
height = 41 instead of 0).
Opening this test document (bnc865381.docx) with following patch included or
excluded looks exactly the same (will attach a screenshot in a second). Left is
as it now is, right is with patch bellow applied.

diff --git a/writerfilter/source/dmapper/TablePropertiesHandler.cxx
b/writerfilter/source/dmapper/TablePropertiesHandler.cxx
index 0edbd71..0c97602 100644
--- a/writerfilter/source/dmapper/TablePropertiesHandler.cxx
+++ b/writerfilter/source/dmapper/TablePropertiesHandler.cxx
@@ -103,9 +103,9 @@ namespace dmapper {
                         // In case a cell already wanted fixed size, we should
not overwrite it here.
                         if (!pManager || !pManager->IsRowSizeTypeInserted())
                             pPropMap->Insert( PROP_SIZE_TYPE, uno::makeAny(
pMeasureHandler->GetRowHeightSizeType() ), false)
-
+                                       }
                         pPropMap->Insert( PROP_HEIGHT,
uno::makeAny(pMeasureHandler->getMeasureValue() ));
-                    }
+                    //}
                     insertRowProps(pPropMap);
                 }
             }

Any idea?

Thanks in advance,
Joren

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to