Issue 82456 - Default Asian font setting is not correct for New Chart
Summary: Default Asian font setting is not correct for New Chart
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
URL:
Keywords: CJK, regression
Depends on:
Blocks: 72764
  Show dependency tree
 
Reported: 2007-10-10 08:24 UTC by naoyuki
Modified: 2013-02-24 21:22 UTC (History)
8 users (show)

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

Attachments
patch_080303 (9.19 KB, text/plain)
2008-03-03 07:15 UTC, weiz 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description naoyuki 2007-10-10 08:24:38 UTC 
How to reproduce:
1. Open Calc with Asian language support ON
2. Insert->Chart.. and click Finish on Chart Wizard
3. Format->Legend and go to Character tab
4. Language list for both of Western and Asian are blank and Font entry for
Asian is inconsistent with
Cell font properties (Font tab in Format->Cell dialog)

For instance, Asian font in Japanese StarSuite8 chart must be "HG PゴシックB
Sun", but Asian chart font is "Andale Sans UI" which is old (StarSuite7 era)
default font. 
This is regression from 2.2.1.

Stefan, please re-assign this to appropriate engineer, thanks!
Comment 1 stefan.baltzer 2008-01-21 16:16:32 UTC 
SBA->BM: Please proceed, thx.
Comment 2 bjoern.milcke 2008-01-22 08:59:09 UTC 
Adding weiz on CC list.

->weiz: I suppose the same applies to OOo with an Asian locale. (Chinese or
Japanese?) Have a look at chart2/source/tools/CharacterProperties.cxx. In method
CharacterProperties::AddDefaultsToMap() the defaults for all (most) character
properties are set.
Comment 3 bjoern.milcke 2008-01-22 09:01:31 UTC 
Hm, /me is in doubt about "HG PゴシックB Sun". I can't read the Japanese
characters, but the word "Sun" is included, so this is probably StarSuite only.

->SBA: Is this a plain StarSuite issue, or does it also apply to OOo?
Comment 4 bjoern.milcke 2008-02-20 08:26:11 UTC 
Probably similar to Issue 85934 for western fonts.
Comment 5 weiz 2008-03-03 07:14:00 UTC 
->bm: The patch is finished, but it's only for fixing the problems as this 
issue described. 
About the font preview problem, I am trying to find a suitable way to fix it, 
but it's not finished yet. I think it's better to submit the patch of font 
preview to corresponding issue( issue 26572 ). :-)
Comment 6 weiz 2008-03-03 07:15:07 UTC 
Created attachment 51858 [details]
patch_080303
Comment 7 IngridvdM 2008-03-07 16:13:24 UTC 
->weiz, I took a part of the patch for the language local to fix issue 84964.
But I removed the hard coded languages and used that one from the configuration
instead.
The font defaults needed a correction also.  I changed them to use the
configuration locale and the spreadsheet font default.
Comment 8 IngridvdM 2008-03-07 16:15:00 UTC 
Fixed in CWs chart23.
Comment 9 IngridvdM 2008-03-10 17:18:52 UTC 
Comparing the implementation of the old and the new chart,
I changed the defaults for font family, stylename,  charset and pitch also to be
taken from the default font. I cannot see any visible effect right now for OOo,
but I assume that this is more correct.
Comment 10 IngridvdM 2008-03-13 15:58:23 UTC 
->Thomas, please verify in CWS chart23.
The default font is now that one which is used by the calc dependent on the
systems locale as it was before in the old charts implementation.
To reproduce set the asian default language to japanese under
Tools->Options->Language Settings->Languages and create a chart then.
Comment 11 kla 2008-04-03 13:02:18 UTC 
Seen ok in CWS chart 23 -> verified
Comment 12 kla 2008-07-03 14:10:28 UTC 
Seen ok in current master -> closed