Bug 276188 - pptx: Text and Euqation symbols in the attached 2010 document are overlapping.
Summary: pptx: Text and Euqation symbols in the attached 2010 document are overlapping.
Status: ASSIGNED
Alias: None
Product: calligrastage
Classification: Applications
Component: filter/pptx (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Matus Uzak
URL:
Keywords:
Depends on: 282617
Blocks:
  Show dependency treegraph
 
Reported: 2011-06-21 14:03 UTC by keerthi
Modified: 2011-11-15 11:24 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Testdata. (103.53 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2011-06-21 14:06 UTC, keerthi
Details
Screenshot in calligrastage. (207.82 KB, image/x-png)
2011-06-21 14:09 UTC, keerthi
Details
screenshot in MSoffice 2010. (71.40 KB, application/x-zip-compressed)
2011-06-21 14:13 UTC, keerthi
Details

Note You need to log in before you can comment on or make changes to this bug.
Description keerthi 2011-06-21 14:03:07 UTC
Version: git checkout fc64fbad1f4847add90ceb6f303b2e383743cfd6
        (Using KDE 4.5.5)                
     OS: Kubuntu 10.10


Test data is in MS PowerPoint 2010 format which has the text and equations in
it.
Open the same document in calligra stage and check the display of the text and
equations.

Expected outcome:
Text and Equations should be displayed properly.

Actual outcome:
Text and Equation symbols are overlapping.

Please check the screenshots attached.

Frequency:
Always.
Comment 1 keerthi 2011-06-21 14:06:20 UTC
Created attachment 61209 [details]
Testdata.
Comment 2 keerthi 2011-06-21 14:09:11 UTC
Created attachment 61210 [details]
Screenshot in calligrastage.
Comment 3 keerthi 2011-06-21 14:13:02 UTC
Created attachment 61211 [details]
screenshot in MSoffice 2010.
Comment 4 Matus Uzak 2011-08-17 09:25:01 UTC
I remember fixing the same issue/file in libmso, checking the ODP for differences.
Comment 5 Matus Uzak 2011-08-17 14:46:24 UTC
Comments to the 2nd slide:

Padding inside frames does not work at the moment.  Moreover fo:padding-top is not set properly for any of the frames.  Also the fo:margin-top attribute of paragraph styles is set to smaller value.  As a result of these issues the 2nd equation does not fit into the space between the paragraphs.
Comment 6 Matus Uzak 2011-08-17 14:52:54 UTC
Git commit e2525645c2636aa41757cd471567239d0a983dba by Matus Uzak.
Committed on 17/08/2011 at 16:48.
Pushed by uzak into branch 'master'.

libmsooxml: Store internal margin info as fo:padding-* instead of fo:margin-*.

Sync with libmso.  Checked with ODFs produced by OOo and MS Office 2007.

CCBUG:276188

M  +4    -4    filters/libmsooxml/MsooXmlCommonReaderDrawingMLImpl.h

http://commits.kde.org/calligra/e2525645c2636aa41757cd471567239d0a983dba
Comment 7 Matus Uzak 2011-08-19 11:46:25 UTC
Git commit 1f50f28428ceca0e7b7b85f9b98e169e89914051 by Matus Uzak.
Committed on 19/08/2011 at 13:40.
Pushed by uzak into branch 'master'.

pptx: Typo--

* Now we have correct padding inside frames.

CCBUG:276188

M  +12   -6    filters/stage/pptx/PptxXmlSlideReader.cpp

http://commits.kde.org/calligra/1f50f28428ceca0e7b7b85f9b98e169e89914051
Comment 8 Matus Uzak 2011-08-19 17:02:21 UTC
Git commit a87ebca1b3563c7727681430fc1a28e7f68ea89d by Matus Uzak.
Committed on 19/08/2011 at 18:51.
Pushed by uzak into branch 'master'.

pptx: Updated calculation of vertical paragraph margins.

MS PowerPoint ususes a specific approach to calculate paragraph margins when
paragraph spacing is provided as a percentage value.  Using the same approach
as in the PPT filter.

We are now compatible with the ODP output filter from MS PowerPoint2007.
The caclulated values equal to ceiled values from the MS PowerPoint2007 UI.

CCBUG:276188

M  +31   -6    filters/libmsooxml/MsooXmlCommonReaderDrawingMLImpl.h
M  +2    -1    filters/libmsooxml/MsooXmlCommonReaderDrawingMLMethods.h
M  +16   -0    filters/stage/pptx/PptxXmlSlideReader.cpp
M  +10   -1    filters/stage/pptx/PptxXmlSlideReader.h

http://commits.kde.org/calligra/a87ebca1b3563c7727681430fc1a28e7f68ea89d
Comment 9 Matus Uzak 2011-08-22 16:29:38 UTC
Git commit e389aa4c0519af47f89e133235ccb351deee3683 by Matus Uzak.
Committed on 22/08/2011 at 18:24.
Pushed by uzak into branch 'master'.

libmsooxml: Updated processing of list styles.

* Store the left margin and indent information into the <style:list-level-label-alignment> element
  instead of storing in into paragraph-properties of the paragraph.  This result in better positioning
  of the label.

CCBUG:276188

M  +13   -11   filters/libmsooxml/MsooXmlCommonReaderDrawingMLImpl.h

http://commits.kde.org/calligra/e389aa4c0519af47f89e133235ccb351deee3683