Issue 86579

Summary: Can't change Asian fonts in Charts
Product: General Reporter: yaw <yaw>
Component: chartAssignee: kla <thomas.klarhoefer>
Status: CLOSED FIXED QA Contact: issues@graphics <issues>
Severity: Trivial    
Priority: P2 CC: IngridvdM, issues, kamataki, masaya.k, tora3, weizhao
Version: 3.3.0 or older (OOo)Keywords: regression
Target Milestone: ---   
Hardware: PC   
OS: Windows 2000   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 72764    
Attachments:
Description Flags
How it happens and looks.
none
sample spreadsheets
none
patch_080305
none
Patch-applied dll files for OpenOffice.org 2.3 Windows
none
Shared library files for Windows incorporating cws chart23 for OpenOffice.org
none
Shared library files for Linux incorporating cws chart23 for OpenOffice.org none

Description yaw 2008-02-29 15:32:36 UTC
There's no way to change the Asian text font in Charts.
In the property window, I can change which font to use, but nothing changes with
that settings.
Asian characters are always displayed with something Serif font.

I made an article to explain how it happens and looks, please see the attached file.
Comment 1 yaw 2008-02-29 15:44:20 UTC
Created attachment 51824 [details]
How it happens and looks.
Comment 2 yaw 2008-02-29 15:47:11 UTC
Created attachment 51825 [details]
sample spreadsheets
Comment 3 yaw 2008-02-29 15:49:35 UTC
To see the sample spreadsheets properly, you need Japanese fonts.
To see what happens and how it looks, please open the pdf file. (you don't need
Japanese fonts to see this file.)
Comment 4 yaw 2008-02-29 16:28:58 UTC
I've forgot to choose the platform and OS.
This occurs with Windows2000SP4 and WindowsXP. (And maybe others, too.)
I don't have Linux and UNIX environment.
Comment 5 kla 2008-03-03 12:14:18 UTC
@ Iha: pls have a look.
Comment 6 IngridvdM 2008-03-04 09:16:50 UTC
Confirmed
->Weizhao, please have a look at this one, thanks!
Comment 7 weiz 2008-03-05 02:05:49 UTC
->iha, the final attachment is the patch for fixing this problem, and it's also 
for CTL fonts.
Comment 8 weiz 2008-03-05 02:07:03 UTC
Created attachment 51888 [details]
patch_080305
Comment 9 IngridvdM 2008-03-05 09:01:33 UTC
->weiz, thanks for the patch, it works fine! I checked it in to CWS chart23. In
addition I changed the order of the items thus they became alphabetically sorted
again.
Comment 10 tora3 2008-03-10 07:05:26 UTC
Created attachment 51998 [details]
Patch-applied dll files for OpenOffice.org 2.3 Windows
Comment 11 tora3 2008-03-10 07:25:00 UTC
tora->iha: Thank you for the fix. It works well.

chart2_bugfix_i86579.zip includes dll files and their source files that 
have been prepared by merging the CWS chart23 into source files of 
OpenOffice.org 2.3 on Mon Mar 10 13:27 2008

  cd $SRC_ROOT/chart2
  cvs update -j CWS_SRC680_CHART23_ANCHOR -j cws_src680_chart23
  cvs update -C -r cws_src680_chart23 source/inc/PropertyHelper.hxx
  cvs update -C -r cws_src680_chart23 source/tools/PropertyHelper.cxx

src/chart2:
  source/controller/itemsetwrapper/CharacterPropertyItemConverter.cxx
  source/controller/main/ChartController_Position.cxx
  source/controller/main/ChartController_Window.cxx
  source/controller/main/PositionAndSizeHelper.cxx
  source/tools/CharacterProperties.cxx
  source/tools/makefile.mk
  source/view/main/PropertyMapper.cxx
  source/inc/PropertyHelper.hxx
  source/tools/PropertyHelper.cxx

builds/windows:
  chartcontroller680mi.dll
  chartmodel680mi.dll
  charttools680mi.dll
  chartview680mi.dll

Those files would be for some users who cannot wait for OOo 3.0. 
Comment 12 IngridvdM 2008-03-10 17:13:48 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 13 IngridvdM 2008-03-10 17:18:16 UTC
Sorry, above comment was meant to be for another issue 82456.
Comment 14 yaw 2008-03-10 17:58:25 UTC
-> All
Thanks a lot for the patches and the DLLs.
The DLLs seem to work well, I'll wait for 3.0 with them.
Comment 15 IngridvdM 2008-03-13 15:27:11 UTC
->Thomas, please verify in CWS chart23.
Comment 16 kla 2008-04-03 09:15:36 UTC
Seen ok in CWS chart 23 -> verified
Comment 17 tora3 2008-04-29 22:06:19 UTC
Because of demand on bug fix, shared library files incorporating cws chart23 
have been built for OpenOffice.org 2.4.0.

http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Id=6485&logon=true&OpenOnly=false&Section=All

============
Installation
============

(Windows)
  Copy build\windows\chart2\*.dll to %PROGRAMFILES%\OpenOffice.org 2.4\program
  Copy build\windows\svx\*.dll    to %PROGRAMFILES%\OpenOffice.org 2.4\program

(Linux)
  Copy build/linux/chart2/*.so.1.1 to /opt/openoffice.org2.4/program
  Copy build/linux/svx/*.so.1.1    to /opt/openoffice.org2.4/program

=============
List of files
=============

  cws_src680_chart23_for_OOo_2.4.0
  |-- README.txt
  |-- build
  |   |-- linux
  |   |   |-- chart2
  |   |   |   |-- libchartcontroller680li.so.1.1
  |   |   |   |-- libchartmodel680li.so.1.1
  |   |   |   |-- libcharttools680li.so.1.1
  |   |   |   `-- libchartview680li.so.1.1
  |   |   `-- svx
  |   |       |-- libcui680li.so.1.1
  |   |       |-- libsvx680li.so.1.1
  |   |       `-- libtextconversiondlgs680li.so.1.1
  |   `-- windows
  |       |-- chart2
  |       |   |-- chartcontroller680mi.dll
  |       |   |-- chartmodel680mi.dll
  |       |   |-- charttools680mi.dll
  |       |   `-- chartview680mi.dll
  |       `-- svx
  |           |-- cui680mi.dll
  |           |-- svx680mi.dll
  |           `-- textconversiondlgs680mi.dll
  `-- src
      |-- chart2.diff
      |-- i36782.diff
      `-- svx.diff
Comment 18 tora3 2008-04-29 22:19:18 UTC
Created attachment 53268 [details]
Shared library files for Windows incorporating cws chart23 for OpenOffice.org
Comment 19 tora3 2008-04-29 22:21:34 UTC
Created attachment 53269 [details]
Shared library files for Linux incorporating cws chart23 for OpenOffice.org
Comment 20 kla 2008-07-18 13:58:16 UTC
Seen ok in current master > closed