Bug 260075

Summary: 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)
Product: [Applications] calligrawords Reporter: Edney Matias <edneymatias>
Component: opendocumentAssignee: Sebastian Sauer <sebsauer>
Status: RESOLVED FIXED    
Severity: normal CC: mail
Priority: NOR    
Version: 1.9.8   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: ODT File
ODT-document that shows the automatic foreground problem

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