Created attachment 189220 [details] Colors only determined by complex-color attributes Open attached file. It contains a shape, for which fill and stroke color is only determined by loext:stroke-complex-color and loext:fill-complex-color. The import reads it correctly into the FillColorTheme, FillColorLumMod, FillColorLumOff, FillComplexColor, and LineComplexColor properties. And the Theme property of the master page is correct too. But the shape is rendered with default color. If a complex color is present, then it has to override the old fill and line color property, respectively. Fill and line color could be created from complex color on import. But I'm not sure, whether that would the best possible solution. Should fill and line color be re-created when ever the assigned theme color changes or the theme itself changes? Or should rendering be changed to use the theme color in the first place? @Tomaž: What do you think?
Currently wrong loext:theme-type is written, see bug 157034. Before the import problem here is fixed to use theme colors instead of draw:fill-color, the export needs to write the correct loext:theme-type.
Hm... there is no svg:stroke-color and draw::fill-color attributes in the document. For backwards compatibility they should always be written in addition to stroke-complex-color and fill-complex-color elements. I have set the same theme to the document again, so the colors are recalculated and the svg:stroke-color and draw::fill-color are properly written to the saved document, so the document also loads with correct colors set. On load we don't trigger theme recalculation, because of the assumption that the old color attributes are still present and have the correctly recalculated theme color set.
(In reply to Tomaz Vajngerl from comment #2) > On load we don't trigger theme recalculation, because of the assumption that > the old color attributes are still present and have the correctly > recalculated theme color set. OK, then lets postpone a solution for the import problem till theme colors are available in an ODF version.
Quikee, Regina: should this be set to "new" until it can be actioned when ODF integrates theme palettes? Or did I misunderstand?