Issue 31097 - Data Point Symbol size changes when a different symbol is choosen
Summary: Data Point Symbol size changes when a different symbol is choosen
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P4 Trivial (vote)
Target Milestone: ---
Assignee: kla
QA Contact: issues@graphics
Keywords: usability
Depends on:
Blocks: 72764
  Show dependency tree
Reported: 2004-07-05 10:38 UTC by kla
Modified: 2013-02-24 21:22 UTC (History)
5 users (show)

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

The patch file of about issue31097#-v1 (1.22 KB, text/plain)
2007-09-29 04:52 UTC, yonggang.mao
no flags Details
The patch file of about i31097#-v2 (2.69 KB, text/plain)
2007-10-17 05:00 UTC, yonggang.mao
no flags Details
The patch file of about i31097#-v3 (3.99 KB, text/plain)
2007-10-22 10:20 UTC, yonggang.mao
no flags Details
The patch file of about i31097#-v4 (5.61 KB, text/plain)
2007-10-24 09:23 UTC, yonggang.mao
no flags Details
The patch of about i31097# (4.51 KB, text/plain)
2007-10-30 15:54 UTC, yonggang.mao
no flags Details
The patch file of about i31097#-v6 (4.13 KB, text/plain)
2007-11-06 06:14 UTC, yonggang.mao
no flags Details
The patch file of about i31097#-v7 (4.14 KB, text/plain)
2007-11-09 03:59 UTC, yonggang.mao
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description kla 2004-07-05 10:38:30 UTC
Changes of Datapoints change also the Size disproportional of the Datapoints.
Comment 1 kla 2004-07-05 10:39:33 UTC
set Keyword
Comment 2 IngridvdM 2004-07-05 15:27:21 UTC
Comment 3 kla 2004-08-11 09:57:48 UTC
according to
this issue will be set to OOoLater
Comment 4 IngridvdM 2005-09-06 14:16:05 UTC
set target for new chart issues
Comment 5 IngridvdM 2006-03-21 18:53:43 UTC
changed summary to indicate a 'new chart' issue
Comment 6 IngridvdM 2006-03-22 11:31:13 UTC
changed issue type from defect to task to better reflect that this issue is only
a working issue for the chart reimplementation
Comment 7 bjoern.milcke 2006-06-23 10:02:32 UTC
Changed target to 2.x
Comment 8 bjoern.milcke 2006-07-03 09:48:48 UTC
Taking over for milestone 8
Comment 9 bjoern.milcke 2006-07-31 12:19:54 UTC
->kla: Either this issue is fixed, or I don't understand it. Please provide some
more comments, attach some screen shots or a bug-document. Thanks.

What changes of datapoints lead to incorrect size changes of data points? And
how are those changes visible (larger, smaller?)
Comment 10 bjoern.milcke 2006-07-31 12:32:10 UTC
Ok, found out what the problem is: If you select a different symbol for the data
point, the size is overwritten by a default size (probably always 1,03cm).
Comment 11 bjoern.milcke 2006-07-31 12:33:02 UTC
chaged subject.
Comment 12 bjoern.milcke 2006-07-31 13:20:17 UTC
This is no regression over the old chart. The behaviour is the same in the old
implementation. Only the default size for changed symbols (not bitmaps) is
"0,21cm" in the old chart and "1,03cm" in the new chart. That's the only difference.
Comment 13 IngridvdM 2006-12-01 22:05:51 UTC
changed prio
Comment 14 IngridvdM 2006-12-01 22:07:17 UTC
changed target
Comment 15 IngridvdM 2007-07-27 13:25:43 UTC
changed type to defect and made summary more clear
Comment 16 bjoern.milcke 2007-09-04 10:38:14 UTC
How to reproduce:
1. Create a line chart with symbols
2. Change the symbol size of a series (e.g. make it 1cm large)
3. Select a different symbol from the "Symbols" menu (last entry in the drop-down)
=> The size is reset to 0,25cm.
Also, when you select "Automatic", you get a different size, this should also
not happen.

Note, when choosing a bitmap (Galery or From File), it is correct that the size
changes, as the bitmaps have different sizes and should not be scaled
unnecessarily per default.

Technical Note: I think the problem might be the brush-item that contains a size
together with the symbol. The dialog that handles symbols is in svx. It is used
by chart and for bullets in writer. Some code is only for chart, there should be
comments about that.
Comment 17 yonggang.mao 2007-09-29 04:52:43 UTC
Created attachment 48599 [details]
The patch file of about issue31097#-v1
Comment 18 yonggang.mao 2007-09-29 05:06:51 UTC
Hello Bjoern,
The patch file only fix the "symbols" menu, however the "automatic" menu have 
not got to fix, please review it, if it has any problem,please tell me.thanks!
Comment 19 yonggang.mao 2007-10-12 07:09:07 UTC
Hello bm,
Please review the patch,thanks!
Comment 20 bjoern.milcke 2007-10-12 16:36:15 UTC
->iha: Would you be so kind as to have a look at the patch, as I am on vacation
the next week. Thanks
Comment 21 IngridvdM 2007-10-15 17:40:30 UTC
I've tested the patch and there are still problems.
If you switch to 'No Symbol' and then back to one of the standard symbols the
size is still zero. Furthermore I think switching to 'Automatic' should not
change the size also.
maoyg, please consider the following:
The boolean bDontSetSize indicates when the size will be modified while changing
the symbol type.
The value of this boolean is wrong in some cases.
The size should only be changed when a symbol from file or from the gallery is
selected. Because these symbols have a size by themselves and should get their
original size on selection. In all other cases the size should not be changed -
even not when 'No Symbol' is selected.
As the naming is very confusing I suggest to replace the variable bDontSetSize
with a variable bResetSize, which has opposite meaning then. The default of the
new variable should be bResetSize=FALSE. And only for the above mentioned cases
it should be set to true.
Comment 22 yonggang.mao 2007-10-17 05:00:15 UTC
Created attachment 48951 [details]
The patch file of about i31097#-v2
Comment 23 yonggang.mao 2007-10-17 05:04:28 UTC
Hello iha,
Please review the patch,thanks.
If there are any problems about it,please tell me,asap! :)
Comment 24 IngridvdM 2007-10-17 18:37:44 UTC
Hi maoyg,
That looks better. But there are still scenarios which are problematic with the
new behavior:
1) When a user starts to choose a symbol from file and aborts the dialog with
cancel then the size is set to zero (this should be avoided). It is wrong to
change anything in that case I think. So return from the method when you get no
graphic there.
2) If an image from gallery or file was set before with a size with an aspect
ratio different from 1:1 and the user now switches to standard symbol, the
standard symbol looks broken as the aspect ratio does not fit. I think we need
this additional logic: If the user switches from gallery or image to standard
symbol and the aspect ratio is different from 1:1 then the size must be adapted
still. Please reset the size in those cases to a mean value of width and heigh.
2b) If we need this complex logic for the standard symbols anyhow, I think we
should use it for the automatic symbol also. The old code line 
"aSymbolSize=aAutoSymbolGraphic.GetPrefSize();" ignores the variable bResetSize
badly anyhow. So I would suggest to remove it. Please have an eye on the
variable 'bNewSize' here also. For me it looks wrong that it is set to false.
Comment 25 yonggang.mao 2007-10-22 10:20:21 UTC
Created attachment 49065 [details]
The patch file of about i31097#-v3
Comment 26 yonggang.mao 2007-10-22 10:24:04 UTC
Hello iha,
I think the variable 'bNewSize' in case MN_SYMBOLS_AUTO is useless,please 
review the patch,thanks! 
Comment 27 IngridvdM 2007-10-22 21:37:02 UTC
Hi Mao, the second problem is not solved.
Comment 28 yonggang.mao 2007-10-24 09:23:21 UTC
Created attachment 49110 [details]
The patch file of about i31097#-v4
Comment 29 yonggang.mao 2007-10-24 09:25:11 UTC
Hello iha and bm,
I have finished the patch,the second problem have been solved.
Please review the patch,thanks!
Comment 30 yonggang.mao 2007-10-30 15:54:14 UTC
Created attachment 49273 [details]
The patch of about i31097#
Comment 31 yonggang.mao 2007-10-30 15:55:52 UTC
Hello iha,
I have removed the variable "bFirstSize".Please review the patch,thanks.
Comment 32 IngridvdM 2007-10-31 14:25:35 UTC
The last patch does not compile on recent master version (e.g. SRC680m235) as
the patch contains compiler warnings which are treated as errors.
Furthermore I think the new use of the variable bNewSize is bad design. We have
the variable bResetSize to manage resizing already. The parallel use of a very
similar mechanism only introduces complete confusion.
Testing the behavior of the new patch we now have the problem, that the size is
again changed when switching from standard to standard symbol in case the symbol
has had a different aspect ration than 1:1.
Please try again, without using the variable bNewSize.
Instead remember the old symboltype at the beginning:
long nPreviousSymbolType=nSymbolType;
Then reset the size dependent on from which old to which new symbol type the
user switches.
Comment 33 yonggang.mao 2007-11-06 06:14:45 UTC
Created attachment 49443 [details]
The patch file of about i31097#-v6
Comment 34 yonggang.mao 2007-11-09 03:59:25 UTC
Created attachment 49543 [details]
The patch file of about i31097#-v7
Comment 35 yonggang.mao 2007-11-09 04:01:06 UTC
Hello iha,
This problem has been solved on the patch i31097#-v7.Please review it,thanks!
Comment 36 IngridvdM 2007-11-16 11:04:36 UTC
Hi maoyg, thanks for the patch!
It is committed to CWS chart15.
Comment 37 IngridvdM 2007-11-16 11:06:08 UTC
Fixed in CWS chart15.
Comment 38 IngridvdM 2007-12-13 17:09:03 UTC
->Thomas, please verify in CWS chart15.
Comment 39 kla 2007-12-28 07:54:59 UTC
Works fine for all Datapoints -> verified
Comment 40 kla 2008-06-13 07:41:09 UTC
seen ok in current master -> closed