Please note that this issue is mainly about "corruption", not long opening time. The open time has been largely addressed in 3.6, and is still worked upon in Bug 44736, while the problem with wrong character background color stays.

Apparently the RTF importer ignores \chcbpat0 control word. In the color table of this file, the color 0 is skipped (the \colortbl control word is immediately followed by semicolon); the RTF standard specifies this as "use the default color as color 0" (Word 2007: Rich Text Format (RTF) Specification, version 1.9.1 (http://www.microsoft.com/download/en/details.aspx?id=10725), page 23). But even if I manually define a value for color 0, LO still ignores it. The only way to change the character background (without exiting a group) is to use \chcbpatN, where N>0.

* Forgot to mention that the "default color" depends on the context: the default color for text is usually black; while the default for background is usually white. Generally I would expect the default to be the color of this class in the parent element (say, the background of paragraph) or the color of this element style (not the automatic style that has been created when the color has been applied in the previous step).

(In reply to comment #1)
> Please note that this issue is mainly about "corruption", not long opening
> time. The open time has been largely addressed in 3.6, and is still worked
> upon in Bug 44736, while the problem with wrong character background color
> stays.

Therefore adapted Summary. Added keyword and whiteboard tag.

@ Miklós:

This bug report about wrong character background = highlight color in a RTF file is interesting, because Mike Kaganski has already provided some thoughts about where exactly the bug is (RTF importer ignores \chcbpat0 control word). Maybe this helps to fix this?

Thank you very much!


Still reproducible with
* LibreOffice 3.6.2.1 and
* current master build (LOdev), pull time 2012-09-24
both on Mac OS X 10.6.8 (x86-64).

Mike,

Reproduced thanks. Here is a minimal testdoc to trigger the problem:

{\rtf1
{\colortbl;\red228\green228\blue228;}
before \chcbpat1 highlighted\chcbpat0  after
\par
}

According to the spec, "after" should not be highlighted, but in Writer, it is.

Roman,

As usual, thanks for the keywords.

Miklos

Miklos Vajna committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=91f02521b827bf51f04befd45c51736fe4072fba

fdo#52475 fix import of RTF_CHCBPAT with value 0



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

Resolved in master, -3-6 cherry-pick request: https://gerrit.libreoffice.org/740

Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-3-6":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=52b6ab689bbeb2611af84b694a98282e58426345&g=libreoffice-3-6

fdo#52475 fix import of RTF_CHCBPAT with value 0


It will be available in LibreOffice 3.6.3.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

VERIFIED as FIXED with latest master build -- LOdev 3.7.0.0.alpha0+, build ID: b966a09, pull time: 2012-10-01 23:33:15, on Mac OS X 10.6.8 (Intel).

Now only short portions of the text show the grey text highlight color, as appropriate.


@ Miklós:
Thank you very much for fixing this!