Created attachment 179977 [details] Sample slide (PPTX) Comparing this slide (created/updated in PowerPoint) between Office 365 and Impress we see three instances where there is extra vertical space between different text formats as shown by Impress. Version: 7.4.0.0.alpha0+ / LibreOffice Community Build ID: e9c50fbbc3b07ef927d133da9cf2395c55611e0f CPU threads: 8; OS: Linux 5.17; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded ...all the way back to Version: 6.4.8.0.0+ Build ID: 99b065ec31d032fc08ab14f66430dac4fef904a5 CPU threads: 8; OS: Linux 5.17; UI render: default; VCL: gtk3; TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:libreoffice-6-4, Time: 2020-10-08_08:57:08 Locale: en-US (en_US.UTF-8); UI-Language: en-US and Version: 7.2.8.0.0+ / LibreOffice Community Build ID: d293877ff029ae7c161ccfbade992485fd92fe75 CPU threads: 8; OS: Linux 5.17; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:libreoffice-7-2, Time: 2022-04-26_20:29:27
Created attachment 179978 [details] Visual comparison Office 365 (left) vs LibreOffice 7.4 (right)
Confirm the bug with Version: 7.3.3.2 / LibreOffice Community Build ID: d1d0ea68f081ee2800a922cac8f79445e4603348 CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: gtk3 Locale: ro-RO (ro_RO.UTF-8); UI: en-US Calc: threaded
I set to Minor because this seems to be dependent on Poppins font and link. If we add some text after the link in MSO, new line is created after the link, before new text, in LO.
Created attachment 180286 [details] Sample slide (PPTX) with font changed to Calibri (In reply to Timur from comment #3) > I set to Minor because this seems to be dependent on Poppins font and link. Hmm, actually not. I changed the font from Poppins to Calibri and the issue still appears. See the attached alternate sample document. Timur, do you mind reconsidering and moving this from minor to normal?
Created attachment 180287 [details] Visual comparison Office 365 LibreOffice 7.4 with font changed to Calibri
Created attachment 180434 [details] Sample slide with font and link changes (PPTX) Here is updated sample , to better track font and link changes.
Created attachment 180435 [details] Visual comparison Office 2016 and LibreOffice 7.4+ for sample with font changeS
Sarper Akdemir committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/9da4a2518eea880a2c80515d1346effa7c8c8c11 related tdf#148966: rework sdr compatibility flags It will be available in 7.6.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.
Sarper Akdemir committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0bb169dcfa14e6db44bc0eaeabe12d0d3b62ffd8 tdf#148966: pptx: workaround for multiline fields followed by linebreaks It will be available in 7.6.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.
The initial reported problem is fixed now. But the additional problems later demonstrated by Timur are not. I have attempted to fix the latter case too. The existing multiline field wrapping is a two part hack. One part in ImpEditEngine::CreateLines the other in ImpEditEngine::Paint. Expanding the hack to fix those cases are not really feasible. Instead a proper refactor of the layouting/handling of fields both in EditMode and Paint are required to properly display and wrap multiline fields.
(In reply to Sarper Akdemir from comment #10) > I have attempted to fix the latter case too. The existing multiline field > wrapping is a two part hack. One part in ImpEditEngine::CreateLines the > other in ImpEditEngine::Paint. Expanding the hack to fix those cases are not > really feasible. Instead a proper refactor of the layouting/handling of > fields both in EditMode and Paint are required to properly display and wrap > multiline fields. I filed bug #154248 to make sure we keep this (and Timur's reproducer).