Bug 260075 - Lists and some headers load wrong (list and headers correct now but font-color in table is not cause we don't handle automatic font-colors)
Summary: Lists and some headers load wrong (list and headers correct now but font-colo...
Status: RESOLVED FIXED
Alias: None
Product: calligrawords
Classification: Applications
Component: opendocument (show other bugs)
Version: 1.9.8
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Sebastian Sauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-18 14:45 UTC by Edney Matias
Modified: 2011-11-13 20:38 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
ODT File (86.08 KB, application/vnd.oasis.opendocument.text)
2009-02-18 14:47 UTC, David Faure
Details
ODT-document that shows the automatic foreground problem (84.42 KB, application/vnd.oasis.opendocument.text)
2011-02-11 15:40 UTC, Sebastian Sauer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Faure 2010-12-16 00:29:44 UTC


---- Reported by edneymatias@gmail.com 2009-02-18 14:45:36 ----

Version:           koffice-kword-1.9.98.6-1.fc10.i386 (using KDE 4.2.0)
Compiler:          gcc-c++-4.3.2-7.i386 
OS:                Linux
Installed from:    Fedora RPMs

The attached files is one of the many files I have that koffice fails to open correctly. Most of them are simple files, so I'll keep sending them so they can be used as test cases.



---- Additional Comments From edneymatias@gmail.com 2009-02-18 14:47:57 ----

Created an attachment
ODT File

ODT File



--- Bug imported by faure@kde.org 2010-12-16 00:31  ---

This bug was previously known as bug 184787.

Imported an attachment (id=54617)

The original submitter of attachment 54617 [details] is unknown.
   Reassigning to the person who moved it here: faure@kde.org.

Comment 1 Sebastian Sauer 2011-01-12 00:15:46 UTC
Lists are loading correct with calligra words.

But at page #2 the text in the table-header with black background color has wrong text-color (black too rather then white).
Comment 2 Sebastian Sauer 2011-01-17 23:13:47 UTC
Fixed with commit f1b137d01c8c09ad0bbf67bec19dbc17d3f3f09f .
Comment 3 Sebastian Sauer 2011-01-17 23:14:34 UTC
Still the wrong report, reopen. Guess I need more cofffeeee :)
Comment 4 Sebastian Sauer 2011-02-10 18:42:53 UTC
The problem is, that we are defining a wrong default font-color if not font-color was defined explicit. In ODF if no font-color was defined then we need to take the background-color into account and either use a black or a white font-color if the background-color is dark or bright.
Comment 5 Sebastian Sauer 2011-02-11 15:40:33 UTC
Created attachment 57160 [details]
ODT-document that shows the automatic foreground problem
Comment 6 Camilla Boemann 2011-11-13 20:38:02 UTC
Git commit 4102d598792bb66c5e11e3a2cb5e27a071714f01 by C. Boemann.
Committed on 13/11/2011 at 21:16.
Pushed by boemann into branch 'master'.

More work on the auto color.

It should work now - famous last words..

BUG:260101
BUG:260075

M  +18   -9    libs/kotext/styles/KoCharacterStyle.cpp
M  +1    -1    libs/textlayout/KoTextDocumentLayout.h
M  +13   -12   libs/textlayout/KoTextLayoutArea_paint.cpp
M  +2    -2    libs/textlayout/KoTextLayoutTableArea.cpp

http://commits.kde.org/calligra/4102d598792bb66c5e11e3a2cb5e27a071714f01