Bug 67347 - FILEOPEN: MS .pptx Textbox with stacked text imported wrong size
Summary: FILEOPEN: MS .pptx Textbox with stacked text imported wrong size
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
3.3.3 release
Hardware: Other All
: medium normal
Assignee: Attila Szűcs
URL:
Whiteboard: interoperability target:24.8.0
Keywords: filter:pptx
: 151102 (view as bug list)
Depends on:
Blocks: ODF-import PPTX-Textbox
  Show dependency treegraph
 
Reported: 2013-07-26 10:31 UTC by Luke
Modified: 2024-02-27 11:58 UTC (History)
15 users (show)

See Also:
Crash report or crash signature:


Attachments
Comparison of PowerPoint vs Impress (1.90 MB, image/png)
2013-07-26 11:01 UTC, Luke
Details
MS .pptx size and position import error (663.60 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2013-07-26 21:23 UTC, Luke
Details
MSO 97 .ppt imports correctly (1.45 MB, application/vnd.ms-powerpoint)
2013-07-31 06:01 UTC, Luke
Details
stripped-down version of this file (36.30 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2014-08-01 04:53 UTC, Luke
Details
stripped-down version showing issue with stacked text (32.19 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2015-03-21 05:47 UTC, Luke
Details
visual comparison before, after, Powerpoint (546.84 KB, image/png)
2024-02-27 10:24 UTC, Attila Szűcs
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Luke 2013-07-26 10:31:41 UTC
This bug was tested LibreOffice v4.1 running in windows 7 and XP. It appears that LibreOffice loses formatting when importing MS .pptx files. I have attached an example.

Steps to reproduce the bug:
1. Open attached Contoso-Presentation.pptx document in Impress
2. Open attached Contoso-Presentation.pptx document in PowerPoint 2007/Office 365
3. Compare the documents. 

Note The "Sales", "CALCULATING PROFIT", and "Highest Revenue" are sized incorrectly resulting in them being displayed incorrectly.
Comment 1 Luke 2013-07-26 10:57:35 UTC
The file I was trying to attach is over the 3MB limit. You can download it from here: https://skydrive.live.com/view.aspx?resid=9FBE63963526EB25!411&app=PowerPoint&authkey=!AN5Q_kKPvn2eWUQ
Comment 2 Luke 2013-07-26 11:01:30 UTC
Created attachment 83028 [details]
Comparison of PowerPoint vs Impress
Comment 3 Cor Nouws 2013-07-26 11:24:19 UTC Comment hidden (obsolete)
Comment 4 Luke 2013-07-26 21:23:40 UTC
Created attachment 83059 [details]
MS .pptx size and position import error

This is only page 2, with all other slide and pictures removed.
Comment 5 Cor Nouws 2013-07-30 20:32:46 UTC
(In reply to comment #4)
> This is only page 2, with all other slide and pictures removed.

Thanks!


In this attachment: look at the two vertical text boxes left and middle. And at the right one.

Thorsten: was it Lubos that could look at such issues?
Comment 6 Luke 2013-07-31 06:01:20 UTC Comment hidden (obsolete)
Comment 7 tommy27 2014-02-15 15:06:27 UTC Comment hidden (obsolete)
Comment 8 Luke 2014-02-19 03:50:03 UTC Comment hidden (obsolete)
Comment 9 tommy27 2014-02-19 06:03:31 UTC Comment hidden (obsolete)
Comment 10 tommy27 2014-02-19 06:19:11 UTC Comment hidden (obsolete)
Comment 11 Luke 2014-02-19 12:59:14 UTC Comment hidden (obsolete)
Comment 12 tommy27 2014-02-19 13:15:24 UTC Comment hidden (obsolete)
Comment 13 tommy27 2014-02-19 13:21:51 UTC
bad fitting of text in textbox reproduced in LibO 3.3.3 too with attachment 83059 [details] 

updated version field to 3.3.3

I have no 3.3.0 available to test but is likely that this issues comes from the beginning of LibO era.
Comment 14 Luke 2014-08-01 04:53:18 UTC
Created attachment 103784 [details]
stripped-down version of this file

reduced the file to just 2 text boxes.
Comment 15 Joel Madero 2014-08-05 23:20:49 UTC Comment hidden (obsolete)
Comment 16 retired 2014-08-13 10:58:49 UTC
Confirmed:4.3.1RC1:OSX
Comment 17 Björn Michaelsen 2014-08-21 12:17:21 UTC Comment hidden (noise)
Comment 18 tommy27 2014-11-21 21:58:48 UTC Comment hidden (obsolete)
Comment 19 Vinaya Mandke 2015-02-15 07:23:17 UTC
Issue with the text-box on second slide: Text is formatted in columns, which is not imported correctly.
Comment 20 Luke 2015-03-21 05:32:59 UTC
@Vinaya Mandke
Good catch. I have created Bug 90140 as a feature request to add textbox column support to Impress. 

This bug should only address the issue the PPTX importer creating over sized text boxs(or undersized text box margins).
Comment 21 Luke 2015-03-21 05:47:27 UTC
Created attachment 114223 [details]
stripped-down version showing issue with stacked text
Comment 22 tommy27 2015-06-18 05:12:00 UTC Comment hidden (obsolete)
Comment 23 tommy27 2015-12-27 21:47:56 UTC Comment hidden (obsolete)
Comment 24 Xisco Faulí 2016-09-14 23:09:22 UTC
Hi Vinaya,
I'm setting this ticket back to NEW as it has been inactive for more than 3
months.
Feel free to assign it back to you if you're still working on this.
Regards
Comment 25 tommy27 2016-10-11 05:36:06 UTC Comment hidden (obsolete)
Comment 26 Timur 2016-10-25 13:48:05 UTC
(In reply to Luke from comment #0)
> Note The "Sales", "CALCULATING PROFIT", and "Highest Revenue" are sized
> incorrectly resulting in them being displayed incorrectly.
Yes, and also: in MSO those are Century Gothic, in LO they are Century Gothic in Format-Character and Liberation Sans in Sidebar-Properties
Comment 27 Luke 2016-11-18 22:36:59 UTC
We already have a stacked text implementation that the chart application uses for axis labels. Could that be used here?
Comment 28 tommy27 2017-07-23 07:53:54 UTC Comment hidden (obsolete)
Comment 29 Xisco Faulí 2018-03-15 16:15:40 UTC Comment hidden (obsolete)
Comment 30 Xisco Faulí 2019-05-15 15:47:35 UTC
Still reproducible in

Version: 6.3.0.0.alpha1+
Build ID: a3e649c3384d19a5ad540c3d65d5f79b66fd9090
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded

However, this doesn't deserve the highest importance status
Comment 31 Regina Henschel 2019-10-02 18:55:21 UTC
The attribute 'style:direction', which allows the value 'ttb' for vertically stacked characters, is only usable with <style:chart-properties> and <style:table-cell-properties>. So using it for text in shapes would need an extension to the ODF standard.
Comment 32 raal 2022-09-21 17:25:25 UTC
*** Bug 151102 has been marked as a duplicate of this bug. ***
Comment 33 Regina Henschel 2024-02-08 18:15:58 UTC
Hi Attila, I have seen you have started work in https://gerrit.libreoffice.org/c/core/+/163121

Some comments:
I think, 'stacked' needs text to be meaningful. Thus perhaps putting it to SdrTextObj might be better than at SdrObject.

You have introduced it as new property. As such it could be combined with writing modes. That looks strange for me. I consider it as a new kind of writing mode. In OOXML 'wordArtVert' and 'wortArtVertRtl' are values of the 'vert' attribute, which holds the writing mode of shapes.

You need not only 'wordArtVert' but 'wortArtVertRtl' too.
Comment 34 Caolán McNamara 2024-02-08 21:01:48 UTC
I'll put the wiki page derived from Regina's work in here as a handy reference https://wiki.documentfoundation.org/Documentation/Text_in_Custom_Shapes

and the MS TextVerticalValues enum:
https://learn.microsoft.com/en-us/dotnet/api/documentformat.openxml.drawing.textverticalvalues?view=openxml-2.8.1
Comment 35 Commit Notification 2024-02-16 12:07:46 UTC
Attila Szűcs committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/f846efa507252b0584d2753a251f2dd99c34541a

tdf#67347 pptx import: stacked text, minimal impl.

It will be available in 24.8.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.
Comment 36 Gerald Pfeifer 2024-02-17 18:34:08 UTC
Looking at bug #151102 and the document from attachment #182595 [details] there,
this looks much, much better.

Just the text is vertically higher than in Impress, see the new screenshot
in attachment #192614 [details] versus the original comparison in attachment #182596 [details].


Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 5a4ab8cb3a3fbf15de11afc5d8876aaa8a7784c9
CPU threads: 12; OS: Linux 6.7; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Comment 37 Commit Notification 2024-02-20 17:38:25 UTC
Attila Szűcs committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/684d63e133fddeca68055a83d85f75bd73319d39

tdf#67347 fix stacked text kern problems

It will be available in 24.8.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.
Comment 38 Commit Notification 2024-02-21 14:13:58 UTC
Attila Szűcs committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/57ac7b73e898cf8889819d7f3beb3608e61b179b

tdf#67347 fix regression stacked text in calc

It will be available in 24.8.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.
Comment 39 Commit Notification 2024-02-27 08:52:12 UTC
Attila Szűcs committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/0079f0e77e74a355d57b24d3a6b6d1a29f45eb79

tdf#67347 pptx import: stacked + horz/vert aligment

It will be available in 24.8.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.
Comment 40 Attila Szűcs 2024-02-27 10:24:40 UTC
Created attachment 192819 [details]
visual comparison before, after, Powerpoint
Comment 41 Timur 2024-02-27 11:50:14 UTC
I also checked original sample, stacked text is good. I set Verified.
When saved, stacked text opens OK in LO and MSO, also alignment is resolved with the last patch. 
Text boxes and alignmnet behave differently, and in LO they cannot be enlarged manually, to change alignment, but typing enlarges them in the direction of text. That seems worth a separate issue. 
There remains yet another an issue of multiline text, but that is a dedicated non trivial fix.
This was stacked text in shape, text box. There are other bugs for stacked or rotated text in table.