Bug Hunting Session
Bug 105795 - FILEOPEN: PPTX table - background & border not imported (partial regression)
Summary: FILEOPEN: PPTX table - background & border not imported (partial regression)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
5.2.0.1 rc
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, filter:pptx
Depends on:
Blocks: ImpressDraw-Tables PPTX
  Show dependency treegraph
 
Reported: 2017-02-06 13:15 UTC by Tobias Burnus
Modified: 2019-11-08 14:59 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
PPTX file (PowerPoint 2007) with table set to style blue gradient + cell border (34.02 KB, application/vnd.openxmlformats-officedocument.presentationml.presentation)
2017-02-06 13:15 UTC, Tobias Burnus
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Burnus 2017-02-06 13:15:59 UTC
Created attachment 130944 [details]
PPTX file (PowerPoint 2007) with table set to style blue gradient + cell border

The attached PPTX file (MS PowerPoint 2007) has in PowerPoint a blue gradient background (of the table - not for each cell) and thin darker blue lines around each cell.

In LO 5.2.0.1, LO 5.3.0.3 and today's MASTER build it is imported without background and border.

In LO 4.3.7.2, the table has at least a blue background (albeit no gradient, no border).
Comment 1 Xisco Faulí 2017-02-06 16:03:37 UTC
Regression introduced in

author	Matus Uzak <matus.uzak@gmail.com>	2016-02-22 14:49:17 (GMT)
committer	Katarina Behrens <Katarina.Behrens@cib.de>	2016-03-04 22:21:21 (GMT)
commit	01157183eadc1e1848cad13950ed67feb62fa62a (patch)
tree	d14dec433c5a55e261fd013bfa18ccd576a47dfc
parent	e0331002d39244cf9c8944fe291d1d009f919eb5 (diff)

PPTX import: Table cell props: Override default graphic style if applicable.
If no fill type is defined for a table cell, then set it to XML_noFill
explicitly, in order to override the fill type defined by the default
graphic style or the graphic style named standard, added by LO and
exported into ODP.

Adding Cc: to Matus Uzak
Comment 2 QA Administrators 2018-10-22 02:48:36 UTC Comment hidden (obsolete)
Comment 3 Tobias Burnus 2018-11-10 12:41:20 UTC
(In reply to Tobias Burnus from comment #0)
> In LO 5.2.0.1, LO 5.3.0.3 and today's MASTER build it is imported without
> background and border.

Still unchanged in LO 6.1
Comment 4 Xisco Faulí 2019-11-08 14:56:57 UTC
I don't think this is a regression, even before the mentioned the gradient in the table was imported as a solid color
Comment 5 Xisco Faulí 2019-11-08 14:59:50 UTC
Xml code:

                    <a:gradFill flip="none" rotWithShape="1">
                      <a:gsLst>
                        <a:gs pos="0">
                          <a:schemeClr val="accent1">
                            <a:tint val="66000"/>
                            <a:satMod val="160000"/>
                          </a:schemeClr>
                        </a:gs>
                        <a:gs pos="50000">
                          <a:schemeClr val="accent1">
                            <a:tint val="44500"/>
                            <a:satMod val="160000"/>
                          </a:schemeClr>
                        </a:gs>
                        <a:gs pos="100000">
                          <a:schemeClr val="accent1">
                            <a:tint val="23500"/>
                            <a:satMod val="160000"/>
                          </a:schemeClr>
                        </a:gs>
                      </a:gsLst>
                      <a:lin ang="2700000" scaled="1"/>
                      <a:tileRect/>
                    </a:gradFill>
                  </a:tcPr>