Issue 117874 - calc 3.3 diagram: changing line size has no effect on legend line size
Summary: calc 3.3 diagram: changing line size has no effect on legend line size
Status: CONFIRMED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: PC Windows XP
: P3 Normal (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL: http://wiki.services.openoffice.org/w...
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-16 10:43 UTC by ps-PitZyclade
Modified: 2013-09-25 20:41 UTC (History)
2 users (show)

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


Attachments
Example of the resulting buggy line graph in 3.3.0 (11.00 KB, image/png)
2011-04-18 08:06 UTC, ps-PitZyclade
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description ps-PitZyclade 2011-04-16 10:43:54 UTC
Changing the line size in a line diagram of calc has no effort on the legend line size.
Saving the resulting diagram and loeding in calc 2.3.1 it is possible to correct diagram and legend with a double click on the diagram. Therefore 3.2.1 is correct working, 3.3 not.
Comment 1 Regina Henschel 2011-04-16 14:07:20 UTC
I agree that the current behavior is not good. It hinders distinction in monochrome charts.

The spec (see URL) has no specification for the line height of the legend key of a line.
In OOo3.4 the height follows the line height up to 0.05cm, which is the upper limit. In OOo2.4.3 the legend key line has the same height as the graph line, even when it was larger than the font height, so that the keys could overlap. That was not the best solution.

In Excel and in Gnumeric the key line height follows the graph line thickness up to about 50% of the font size height. In Excel this is done in steps, in Gnumeric it is smooth.

I think, that we should do it similar to Gnumeric, use directly the graph line thickness up to 50% of the font size height.

Because the current spec says nothing about the height of the legend key of a line, first the spec has to be extended. Therefore I set the issue to enhancement.
Comment 2 Regina Henschel 2011-04-17 20:30:27 UTC
I see
const sal_Int32 nMaxLineWidthForLegend = 50;/*1/100 mm*///todo: make this dependent from legend entry height
in line 90 in chart2/source/view/main/VLegendSymbolFactory.cxx

I see no easy way to make it dependent on font size.

But I suggest to set the constant to 300 instead of 50. With 300 the line in the legend will correspond to the graphic line up to 0,3cm, which will be enough for all "normal" charts, even in impress. On the other hand it will not exceed font height down to 8.5pt, and for smaller font sizes line width larger than 0,3cm is unlikely.

I have tested it in DEVm100 with the old implementation of the legend and I think it would be a good compromise until the line can be made dependent on font size. But I haven't got an actual build with resizable legend to test it there.
Comment 3 niklas.nebel 2011-04-18 07:54:32 UTC
Chart issue...
Comment 4 ps-PitZyclade 2011-04-18 08:06:04 UTC
Created attachment 76402 [details]
Example of the resulting buggy line graph in 3.3.0
Comment 5 Oliver-Rainer Wittmann 2012-06-13 12:19:11 UTC
getting rid of value "enhancement" for field "severity".
For enhancement the field "issue type" shall be used.