Issue 37136 - Bold text not imported properly
Summary: Bold text not imported properly
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: open-import (show other issues)
Version: 680m60
Hardware: All Windows 2000
: P3 Trivial (vote)
Target Milestone: OOo 2.1
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords:
: 15928 (view as issue list)
Depends on:
Blocks:
 
Reported: 2004-11-14 11:27 UTC by alan
Modified: 2006-11-10 17:41 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Example PPT presentation - see slide 3 for misimported bold text (1.04 MB, application/vnd.ms-powerpoint)
2004-11-14 11:29 UTC, alan
no flags Details
Slide 3 as it appears in PowerPoint (51.16 KB, image/jpeg)
2004-11-14 11:30 UTC, alan
no flags Details
Slide 3 as it appears in Impress (51.85 KB, image/jpeg)
2004-11-14 11:30 UTC, alan
no flags Details
Text in normal view (124.30 KB, image/jpeg)
2006-11-09 08:44 UTC, alan
no flags Details
Text in slide show (cut off) (53.50 KB, image/jpeg)
2006-11-09 08:44 UTC, alan
no flags Details
Pared-down version of the first sample doc. Used to create screenshots (576.50 KB, application/vnd.ms-powerpoint)
2006-11-09 08:46 UTC, alan
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description alan 2004-11-14 11:27:49 UTC
In the attached PowerPoint presentation, slide 3, bold text is imported as
regular weight text. See the first two lines of the 3-bullet group, and the left
portion of the third line of that group.
Comment 1 alan 2004-11-14 11:29:26 UTC
Created attachment 19258 [details]
Example PPT presentation - see slide 3 for misimported bold text
Comment 2 alan 2004-11-14 11:30:03 UTC
Created attachment 19259 [details]
Slide 3 as it appears in PowerPoint
Comment 3 alan 2004-11-14 11:30:36 UTC
Created attachment 19260 [details]
Slide 3 as it appears in Impress
Comment 4 wolframgarten 2004-11-15 07:17:31 UTC
Reproducible. Another powerpoint import problem. Reassigned to Sven.
Comment 5 sven.jacobi 2005-05-26 14:10:01 UTC
accepted
Comment 6 alan 2006-08-29 16:20:55 UTC
Sven, have you had a chance to look at this yet?
Comment 7 sven.jacobi 2006-08-30 09:09:37 UTC
Not yet, but I hope to be able to fix this issue soon.
Comment 8 alan 2006-10-05 12:38:42 UTC
ayaniger -> sj:
I've seen that if a string is in a bold font, and it has mixed Hebrew/English
text, the English is imported as bold, and the Hebrew is not. 

If there's still work to be done on this issue, perhaps I can help. (In any
case, I want to become more familiar with the Impress code, particularly the PPT
import code.) Could you tell me where to look to find where the font weight is
set, so I can check why there's a difference between Hebrew and English?
Comment 9 sven.jacobi 2006-10-05 13:36:26 UTC
The code which is responsible for bold characters can be found in
svx/source/svdraw/svdfppt.cxx

I think the problem will be in PPTParaSheet::Read or in
PPTStyleTextPropReader::PPTStyleTextPropReader

In general a single unknown property leads to problems to parse further
outstanding properties.
Comment 10 alan 2006-10-09 13:05:05 UTC
ayaniger->sj:

I created two simple presentations, each with a 4-char bold text object, one in
Hebrew, one in English. The English text was displayed as bold, but the Hebrew
wasn't. But from what I see, the import of the Hebrew document's properties is
OK. I added trace statemnts in PPTParaSheet::Read and in
PPTStyleTextPropReader::PPTStyleTextPropReader, and I didn't see any differences
relating to bold char properties. In both cases, when the following call stack
was executed:

SdrPowerPointImport::ReadObjText
ImplSdPPTImport::ApplyTextObj
SdrPowerPointImport::ApplyTextObj
PPTPortionObj::ApplyTo
PPTPortionObj::GetAttrib,

I get to the case of PPT_CharAttr_Bold in the switch statement, once for each
document. Doesn't this indicate that the Hebrew is in fact recognized as bold?
If so, where might I look next to find the problem?
Comment 11 sven.jacobi 2006-10-09 15:55:14 UTC
sj->ayaniger: While investigating the problem I found the solution, the PPT
import does not set the SvxPostureItem and SvxWeightItem for Asian and Complex
languages . This issue has been fixed now in svdfppt.cxx -r1.142.28.1
cws[impress109].
Comment 12 sven.jacobi 2006-10-25 09:38:48 UTC
sj->cgu: this issue is ready to be verified in cws[impress109]
Comment 13 christian.guenther 2006-10-30 09:57:35 UTC
CGU: Verified in cws impress109
Comment 14 sven.jacobi 2006-10-31 15:03:05 UTC
*** Issue 15928 has been marked as a duplicate of this issue. ***
Comment 15 christian.guenther 2006-11-07 17:14:47 UTC
CGU: Integrated in OOE680m1
Comment 16 alan 2006-11-09 08:42:45 UTC
The fix seems to be incomplete, as it has created a new problem. In the first
slide of the sample document, the lower text gets cut off during the slideshow.
It displays properly if text is not bold. See screenshots. 
Comment 17 alan 2006-11-09 08:44:13 UTC
Created attachment 40452 [details]
Text in normal view
Comment 18 alan 2006-11-09 08:44:51 UTC
Created attachment 40453 [details]
Text in slide show (cut off)
Comment 19 alan 2006-11-09 08:46:15 UTC
Created attachment 40454 [details]
Pared-down version of the first sample doc. Used to create screenshots
Comment 20 christian.guenther 2006-11-10 16:21:17 UTC
CGU: integrated in src680m193
Comment 21 christian.guenther 2006-11-10 16:33:08 UTC
I can reproduce the second behaviour but this is a new bug.
I write a new issue for the new bug.
Comment 22 christian.guenther 2006-11-10 16:33:57 UTC
I close the issue.
Comment 23 christian.guenther 2006-11-10 17:41:13 UTC
I wrote 2 following issues for this.

issue 71432
and
issue 71433