Description: The attached document has the wrong text element layout in Impress Steps to Reproduce: 1.Open document in Impress 2.The position is incorrect 3. Actual Results: Wrong position Expected Results: Correct text location as in MSO Reproducible: Always User Profile Reset: No Additional Info: 927f948c4c86887236a0451545ce0ca20ec2 CPU threads: 8; OS: Windows 10.0 Build 18362; UI render: Skia/Raster; VCL: win; Locale: en-US (en_US); UI-Language: en-US Calc: CL
Created attachment 158963 [details] Said document
Created attachment 158964 [details] result in Impress
Created attachment 158965 [details] result in MSO
We will be submitting a fix
Changes submitted https://gerrit.libreoffice.org/c/core/+/91023
Reproduced in Version: 7.0.0.0.alpha0+ Build ID: fd1cd5522283f279a01d6d673f676a1346e9358b CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3; Locale: en-US (en_US.UTF-8); UI-Language: en-US Calc: threaded and Version: 5.2.0.0.alpha0+ Build ID: 3ca42d8d51174010d5e8a32b96e9b4c0b3730a53 Threads 4; Ver: 4.19; Render: default;
Hi nd101, your patch also affects bug 119187, please, take it into account
Actually this is a regression. It was fine until https://cgit.freedesktop.org/libreoffice/core/commit/?id=cc6ae367280d205a88c6f907b20b84613221ca52. Then the 'Template' placeholder was gone until https://cgit.freedesktop.org/libreoffice/core/commit/?id=2a7119021edea8346e4673d57cac3f051a5d0968, when it started to be displayed at its current position
*** Bug 128547 has been marked as a duplicate of this bug. ***
@Nd101, you patch fixes the document in bug 128547
nd101 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/69b83dc2d3014dd9b18402534e15c937dc082464 tdf#131554 placeholder iteration fails to stop when a match is found It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
nd101 committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/55d423e8f371e914c51f9aa18aacede46355b133 tdf#131554 placeholder iteration fails to stop when a match is found It will be available in 6.4.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Guess it's solved.
Yes, verified in Version: 7.0.0.0.alpha0+ Build ID: 35fc5ef0a759884b24ed8b83cd05702a0fab64cc CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3; Locale: en-US (en_US.UTF-8); UI-Language: en-US Calc: threaded @nd101, thanks for fixing this issue!
Xisco Fauli committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5750be8cf82b6933cb0b44ffe5ad307da1b4a0eb tdf#132282: Revert fix for tdf#131554 It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
As the patch for this fix was reverted - I set the bug status to REOPENED
(In reply to Serge Krot (CIB) from comment #16) > As the patch for this fix was reverted - I set the bug status to REOPENED The fix was reverted but the unittest still passes. See the comment in the commit: 912217285b3058efa54c2336f91fda4efdad6ff0 fixed the root cause of tdf#131554 and 69b83dc2d3014dd9b18402534e15c937dc082464 is no longer needed The unittest still passes
Textbox position is still correct in Version: 7.0.0.0.alpha0+ Build ID: cf36fe5eb41910c26d58fb25e54ccf2e0ee01365 CPU threads: 4; OS: Linux 4.19; UI render: default; VCL: gtk3; Locale: en-US (en_US.UTF-8); UI-Language: en-US Calc: threaded
Xisco Fauli committed a patch related to this issue. It has been pushed to "libreoffice-6-4": https://git.libreoffice.org/core/commit/7df4ebf9a658681d02842b14187722ae8161cd7e tdf#132282: Revert fix for tdf#131554 It will be available in 6.4.5. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Xisco Fauli committed a patch related to this issue. It has been pushed to "libreoffice-6-4-4": https://git.libreoffice.org/core/commit/3b9283880e8d53c5cbc0eabc1c9119ff673b422c tdf#132282: Revert fix for tdf#131554 It will be available in 6.4.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.