Created attachment 178893 [details] Sample slide (PPTX) How to repeat: 1. Open sample PPTX document in LibreOffice 2. Compare with rendering in Office 365 and notice how the "Business..." text is too wide and how the "Enterprise..." text fails to line break and is too wide. Version: 7.4.0.0.alpha0+ / LibreOffice Community Build ID: be1aab8632ead65d75c0436005d3cac7d43b9f02 CPU threads: 8; OS: Linux 5.16; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US
Created attachment 178894 [details] Visual comparison LibreOffice (left) vs Office 365 (right)
I guess this was created, not some SmartArt. Same if ungrouped. Never was fine, improved few times, last time in 6.1. New. Not sure if there's a similar bug, not easy to find.
Created attachment 181692 [details] Compare the distribution of the lines Your proposed patch #3 at https://gerrit.libreoffice.org/c/core/+/136819 doesn't get the distribution of lines correctly. An existing odp-document, which has soft line breaks (Shift+Return) included in the path text, will look very different in a version with your patch. How does others judge this incompatibility? I'm not in principle against such change, but if other agree that we should change the behavior for existing documents, then you will need to be explain the change in the release notes. The "textButton"-shape is one of the shapes, which are available in MS binary formats too. I have not yet tested the behavior in old MS Office. Have you already tested it?
Created attachment 181794 [details] File to reproduce the crash The attached file crashes for me with your patchset 5. Problem in addition: The Shift+Enter problem occurs in all types of WordArt. Why restrict it to the four "Follow Path" types?
Created attachment 184784 [details] Convert line feed to paragraph in import filter I still think, this should be done in the import filter and not in Fontwork rendering. That way existing ODF documents will not change.
Attila Szűcs committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f9d6dd788e82a1964dab9cc0d0436c8c54b775c0 tdf#148000 impress: Handle linebreaks on fontwork. 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.
Created attachment 184790 [details] effect of Attila's patch Before / After screenshot to demonstrate the effect of Attila's patch.
Attila Szűcs committed a patch related to this issue. It has been pushed to "libreoffice-7-5": https://git.libreoffice.org/core/commit/497298874961fb335caf4cc91e531667394588bc tdf#148000 impress: Handle linebreaks on fontwork. It will be available in 7.5.1. 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.
Attila Szűcs committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/3a5cc107efb843b01193230c56dc8d9c0072348a tdf#148000 impress: improve fontwork text placement. It will be available in 24.2.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.
Happy to confirm this as verified with last night's build: Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community Build ID: 9602f8a9318dd4d3409856e2ae06abe96e72b51b CPU threads: 12; OS: Linux 6.6; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US
Attila Szűcs committed a patch related to this issue. It has been pushed to "libreoffice-7-6": https://git.libreoffice.org/core/commit/3d7dad88c409fecd3ba4a3f27f8e2e6e2e5f14d7 tdf#148000 impress: improve fontwork text placement. It will be available in 7.6.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.