Created attachment 142388 [details] Simple reproducer In certain .doc documents the columns are not lining up in Writer like they do in MS Word or Apache OpenOffice. Steps to reproduce: 1. Open AlignmentBug.doc in Writer 2. Compare to MS Word or AOO
Created attachment 142389 [details] LO vs AOO comparison
Version: 4.1.0.1 Build ID: 1b3956717a60d6ac35b133d7b0a0f5eb55e9155 GOOD
I can't compare it with MS Word, but for me it seems, that the page margins have changed. Luke, can you confirm this?
Created attachment 142402 [details] how it looks in MS2010
Ouch, you found your own regression ;-) Regression introduced by: author Luke Deller <luke@deller.id.au> 2014-07-06 23:19:18 +1000 committer Michael Stahl <mstahl@redhat.com> 2014-07-09 10:58:31 +0000 commit f312ef98de57a5926be67421395544bb9d41b809 (patch) tree 4292737b838068de2520340d5e4af20e21b7a679 parent c1b562139a2e87cce2f189c50a242cf057970551 (diff) fix fdo#77844: header wrongly enabled from .doc The LO page style needs page headers to be turned on if the corresponding .doc file section has a left (=even) page header or a first page header. However this should not be triggered in the case where a first page header is present but hidden due to the "different first page" header option being disabled. This case is fixed by this commit. Bisected with: bibisect-44max Adding Cc: to Luke Deller
Actually, you're different Lukes... Adding Cc: to Luke Deller
LOL thanks Xisco & Luke. I see the problem: it is calculating the top margin as if there were a header, but there isn't a header. If you manually edit the top margin to match the other margins (2.54cm) then it looks good again. The document section actually contains a first page header with the text "Hayden Management", but this is not shown because the "Different first page" header setting is disabled. The doc import code to calculate the top margin is confused by the presence of this header.
Luke Deller committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6c68e52532d45c97a83b2396bd40c98ade9b5bb4 Fix top margin in DOC import for tdf#117885 It will be available in 6.2.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.
Verified FIXED in Version: 6.2.0.0.alpha0+ Build ID: 18a8cac57d5450fef9111fa356839f07c7593ad0 Thanks for taking care of this Luke Deller!
Verified in Version: 6.2.0.0.alpha0+ Build ID: 2a7f74900fb646235b74d4c9bd4690e44edc3ed4 CPU threads: 4; OS: Linux 4.13; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded @Luke ( both of you ), thanks for reporting and fixing this!! ;-)
Cherry-picked: 6.1: https://gerrit.libreoffice.org/#/c/55465/1 6.0: https://gerrit.libreoffice.org/#/c/55466/
Luke Deller committed a patch related to this issue. It has been pushed to "libreoffice-6-1": http://cgit.freedesktop.org/libreoffice/core/commit/?id=294725dd76e6c5288d2e96f47c730a94c9b09037&h=libreoffice-6-1 Fix top margin in DOC import for tdf#117885 It will be available in 6.1.0.1. 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.
Hi Luke Deller, Please check the comment from Michael Stahl in https://gerrit.libreoffice.org/#/c/55466/ Regards
(In reply to Xisco Faulí from comment #13) > Please check the comment from Michael Stahl in > https://gerrit.libreoffice.org/#/c/55466/ Yes sorry Xisco, Michael is right. This is the same issue that you reported already in bug 118412, which I fixed in https://gerrit.libreoffice.org/#/c/56602/ I guess we should squash these two together for the backport.
(In reply to Luke Deller from comment #14) > (In reply to Xisco Faulí from comment #13) > > Please check the comment from Michael Stahl in > > https://gerrit.libreoffice.org/#/c/55466/ > > Yes sorry Xisco, Michael is right. This is the same issue that you reported > already in bug 118412, which I fixed in > https://gerrit.libreoffice.org/#/c/56602/ > > I guess we should squash these two together for the backport. Ouch you're right! I missed it Yep, it's a good idea to put both of them together
Luke Deller committed a patch related to this issue. It has been pushed to "libreoffice-6-0": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3db1874044c24dadfb2f273d0e57e15bdcada1f5&h=libreoffice-6-0 Fix top margin in DOC import for tdf#117885 It will be available in 6.0.6. 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.