| Summary: | docx autosize table created very large - extends way past right margin | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Justin L <jluth> |
| Component: | Writer | Assignee: | Justin L <jluth> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | jluth, vmiklos, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 4.4.0.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:5.4.0 target:5.3.0.2 target:5.2.5 | ||
| Crash report or crash signature: | Regression By: | ||
| Attachments: |
sw/qa/extras/ooxmlexport/data/hidemark.docx
hidemark_lo54alpha.pdf: shows why this bug report was created |
||
|
Description
Justin L
2016-12-23 09:23:27 UTC
Created attachment 129891 [details]
hidemark_lo54alpha.pdf: shows why this bug report was created
Moving it to NEW as the problematic commit has been identified. Adding Cc: to Miklos Vajna proposing https://gerrit.libreoffice.org/32385 tdf#104876 writerfilter: m_bTableSizeTypeInserted = false here This is a partial fix which ought to undo the regression. However, the table width is still wrong - it seems to be twice as wide as it should be. I'm guessing that hidemark.docx was a hand-crafted txt document, so perhaps there are some content errors. However, that still suggests some kind of bug in handling the specification of 4 grids, but only two non-spanning cells. If word2013 round-trips hidemark.docx, then we only get two gridCols instead of 4. <w:tblGrid><w:gridCol w:w="960"/><w:gridCol w:w="960"/></w:tblGrid> So it seems to be a consistency problem with the document itself. I'll just ignore that part of the problem since it isn't a completely valid document. Justin Luth committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e73c961a7fb23246dc4a9c27d7ed0808b9ba74b5 tdf#104876 writerfilter: m_bTableSizeTypeInserted = false here It will be available in 5.4.0. 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. Justin Luth committed a patch related to this issue. It has been pushed to "libreoffice-5-3": http://cgit.freedesktop.org/libreoffice/core/commit/?id=91867c343d858e7365c06f2953e979bc038af186&h=libreoffice-5-3 tdf#104876 writerfilter: m_bTableSizeTypeInserted = false here It will be available in 5.3.0.2. 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. Justin Luth committed a patch related to this issue. It has been pushed to "libreoffice-5-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bed818c5d5e92a0b189f25e18495fc205d949128&h=libreoffice-5-2 tdf#104876 writerfilter: m_bTableSizeTypeInserted = false here It will be available in 5.2.5. 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. verified. version: 5.4.0.0.alpha0+ / build id: febc116 / ls-4001 / android 5.1 the first column does not fill entire page & displays two columns. |