Bug 128161 - Save after the file is opened, the ppt line spacing is reduced until 0 (not with PPTX)
Summary: Save after the file is opened, the ppt line spacing is reduced until 0 (not w...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: low minor
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: filter:ppt
Depends on:
Blocks: PPT
  Show dependency treegraph
 
Reported: 2019-10-16 00:42 UTC by xiaohaodaye
Modified: 2022-09-05 17:56 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
Start setting is 300% (124.50 KB, application/vnd.ms-powerpoint)
2019-10-16 00:43 UTC, xiaohaodaye
Details
test.png: 4 screenshots (46.02 KB, image/png)
2019-10-16 01:03 UTC, xiaohaodaye
Details

Note You need to log in before you can comment on or make changes to this bug.
Description xiaohaodaye 2019-10-16 00:42:50 UTC
Description:
ppt line spacing setting, saved as .ppt will be reduced every time the line spacing is saved until 0

Steps to Reproduce:
1.open a file
2.Save and close
3.Open view, reduced line spacing

Actual Results:
Line spacing reduction

Expected Results:
Try to fix the bug


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 xiaohaodaye 2019-10-16 00:43:58 UTC
Created attachment 155027 [details]
Start setting is 300%
Comment 2 xiaohaodaye 2019-10-16 01:03:15 UTC
Created attachment 155029 [details]
test.png: 4 screenshots
Comment 3 Timur 2019-11-14 09:43:07 UTC Comment hidden (obsolete)
Comment 4 Justin L 2020-04-28 08:54:03 UTC
The focus of test.ppt from comment 1 is on the second paragraph(456), which actually starts at linespacing multiple: 280% according to PowePoint 2003 and PowerPoint 2016.
Comment 5 Justin L 2020-04-28 11:42:39 UTC
It looks like the code very deliberately messes this up - based on font scaling.
There is no value in FontIndependentLineSpacing (not sure how this is different from pPara->mbFixedLineSpacing, where elsewhere FontIndependLineSpace >>= bFixedLineSpacing) and therefore everything scales by pDesc->Scaling + 0.5.

Unfortunately, all this code comes from OOo, so it is hard to identify the purpose of it. The code is at least related to this OOo bug - https://bz.apache.org/ooo/show_bug.cgi?id=28747

Since everything imports the same as MS Powerpoint, this must be an export bug. On the other hand, since this adjusts the LO value on export, it ought to do the reverse equation during input. So the first task is to find out why this scaling code was added in the first place.

And with that I walk away since I don't understand font / scaling / presentation stuff.
Comment 6 QA Administrators 2022-04-29 03:44:52 UTC Comment hidden (obsolete)
Comment 7 Timur 2022-04-29 11:16:01 UTC
Repro 7.4+ for PPT not or PPTX.