Issue 91129 - Scale text Button not persistent
Summary: Scale text Button not persistent
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:
Depends on: 93256
Blocks: 72764
  Show dependency tree
 
Reported: 2008-06-27 11:32 UTC by kla
Modified: 2013-02-24 21:22 UTC (History)
3 users (show)

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


Attachments
patch_080725 (4.47 KB, text/plain)
2008-07-25 09:09 UTC, weiz
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description kla 2008-06-27 11:32:28 UTC
Create a Chart
Scale text Button on
Save / reload
Scale Text button switched back to default (off)
Comment 1 weiz 2008-07-25 09:08:08 UTC
@iha: The patch is finished, it's also for fixing the issue 60824. 
The patch is works fine in my computer, but I think may be some name of the 
functions and variables should be adapted. Please let me know your suggestions. 
Thanks a lot!
Comment 2 weiz 2008-07-25 09:09:12 UTC
Created attachment 55356 [details]
patch_080725
Comment 3 IngridvdM 2008-08-11 15:36:31 UTC
@weiz, the patch is not ok. With the patch the problems are bigger than without:
-Create a chart with axis labels and data point labels. Switch on scaling, save,
reload, and double click the chart -> the font sizes got wrong at the axes and
at the data labels.
-Create a chart with titles and legend, ensure that text scaling is switched
off, save, reload and resize the chart -> titles and legend text do scale, where
they should not.
Comment 4 IngridvdM 2008-08-29 15:22:02 UTC
I implemented issue 93256 'don't change font sizes when only the chart wall is
resized'. This reduces the code complexity thus the ScaleText property now can
be saved and loaded correctly without the unsure dependency to a sometimes
unknown diagram size.

Fixed in CWS chart30.
Comment 5 IngridvdM 2008-09-10 16:31:03 UTC
@KLA, please verify in CWS chart30.
Comment 6 kla 2008-09-16 10:11:48 UTC
Seen ok in CWS chart30 -> verified
Comment 7 kla 2008-11-04 12:15:44 UTC
Seen ok in current master -> closed