TAChart: Fix axes for series having very long labels. Issue #34896. Based on patch by Marcin Wiazowski.

git-svn-id: trunk@60129 -
This commit is contained in:
wp 2019-01-20 23:33:19 +00:00
parent 12278e7ef4
commit 3b38aad6ce
3 changed files with 53 additions and 10 deletions

View File

@ -250,8 +250,8 @@ type
FMin, FMax: PDouble; FMin, FMax: PDouble;
function CalcOffset(AScale: Double): Double; function CalcOffset(AScale: Double): Double;
function CalcScale(ASign: Integer): Double; function CalcScale(ASign: Integer): Double;
constructor Init( constructor Init(AAxis: TChartAxis; AImageLo, AImageHi: Integer;
AAxis: TChartAxis; AImageLo, AImageHi, AMarginLo, AMarginHi: Integer; AMarginLo, AMarginHi, ARequiredMarginLo, ARequiredMarginHi: Integer;
AMin, AMax: PDouble); AMin, AMax: PDouble);
procedure UpdateMinMax(AConv: TAxisConvFunc); procedure UpdateMinMax(AConv: TAxisConvFunc);
end; end;
@ -1215,9 +1215,40 @@ end;
{ TAxisCoeffHelper } { TAxisCoeffHelper }
constructor TAxisCoeffHelper.Init( procedure EnsureGuaranteedSpace(var AValue1, AValue2: Integer;
AAxis: TChartAxis; AImageLo, AImageHi, AMarginLo, AMarginHi: Integer; AImage1, AImage2, AMargin1, AMargin2, AGuaranteed: Integer);
var
delta1, delta2: Integer;
begin
delta1 := AImage1 - AValue1;
delta2 := AImage2 - AValue2;
if (AValue2 = AImage2 - AMargin2) then begin
AValue2 := AImage2 - AMargin2;
AValue1 := AValue2 - AGuaranteed;
end else
if (AValue1 = AImage1 + AMargin1) then begin
AValue1 := AImage1 + AMargin1;
AValue2 := AValue1 + AGuaranteed;
end else
begin
AValue1 := (AImage1 + AImage2 + AGuaranteed) div 2;
AValue2 := AValue1 + AGuaranteed;
if AValue1 + delta1 >= AImage1 then begin
AValue1 := AImage1 - delta1;
AValue2 := AValue1 + AGuaranteed;
end else
if AValue2 + delta2 <= AImage2 then begin
AValue2 := AImage2 - delta2;
AValue1 := AValue2 - AGuaranteed;
end;
end;
end;
constructor TAxisCoeffHelper.Init(AAxis: TChartAxis; AImageLo, AImageHi: Integer;
AMarginLo, AMarginHi, ARequiredMarginLo, ARequiredMarginHi: Integer;
AMin, AMax: PDouble); AMin, AMax: PDouble);
const
GUARANTEED_SPACE = 10;
begin begin
FAxis := AAxis; FAxis := AAxis;
FImageLo := AImageLo; FImageLo := AImageLo;
@ -1226,14 +1257,26 @@ begin
FMax := AMax; FMax := AMax;
FLo := FImageLo + AMarginLo; FLo := FImageLo + AMarginLo;
FHi := FImageHi + AMarginHi; FHi := FImageHi + AMarginHi;
if Assigned(AAxis) and AAxis.IsVertical then begin
if (FHi + GUARANTEED_SPACE >= FLo) then
EnsureGuaranteedSpace(FHi, FLo, FImageHi, FImageLo,
ARequiredMarginHi, ARequiredMarginLo, GUARANTEED_SPACE);
end else begin
if (FLo + GUARANTEED_SPACE >= FHi) then
EnsureGuaranteedSpace(FLo, FHi, FImageLo, FImageHi,
ARequiredMarginLo, ARequiredMarginHi, GUARANTEED_SPACE);
end;
end; end;
function TAxisCoeffHelper.CalcScale(ASign: Integer): Double; function TAxisCoeffHelper.CalcScale(ASign: Integer): Double;
begin begin
if (FMax^ = FMin^) or (Sign(FHi - FLo) <> ASign) then exit(1.0); if (FMax^ <= FMin^) or (Sign(FHi - FLo) <> ASign) then
if (FAxis <> nil) and FAxis.IsFlipped then Result := ASign
Exchange(FLo, FHi); else
Result := (FHi - FLo) / (FMax^ - FMin^); Result := (FHi - FLo) / (FMax^ - FMin^);
if (FAxis <> nil) and FAxis.IsFlipped then
Result := -Result;
end; end;
function TAxisCoeffHelper.CalcOffset(AScale: Double): Double; function TAxisCoeffHelper.CalcOffset(AScale: Double): Double;

View File

@ -1427,7 +1427,7 @@ begin
lmpNegative: isNeg := true; lmpNegative: isNeg := true;
lmpInside: isNeg := Source[AIndex]^.Y >= ref; lmpInside: isNeg := Source[AIndex]^.Y >= ref;
end; end;
if (isRotated and ParentChart.IsRightToLeft) xor GetAxisY.Inverted then if (IsRotated and ParentChart.IsRightToLeft) xor GetAxisY.Inverted then
isNeg := not isNeg; isNeg := not isNeg;
Result := DIR[IsRotated, isNeg]; Result := DIR[IsRotated, isNeg];
end; end;

View File

@ -583,10 +583,10 @@ var
begin begin
rX.Init( rX.Init(
BottomAxis, FClipRect.Left, FClipRect.Right, AMargin.Left, -AMargin.Right, BottomAxis, FClipRect.Left, FClipRect.Right, AMargin.Left, -AMargin.Right,
@FCurrentExtent.a.X, @FCurrentExtent.b.X); FMargins.Left, FMargins.Right, @FCurrentExtent.a.X, @FCurrentExtent.b.X);
rY.Init( rY.Init(
LeftAxis, FClipRect.Bottom, FClipRect.Top, -AMargin.Bottom, AMargin.Top, LeftAxis, FClipRect.Bottom, FClipRect.Top, -AMargin.Bottom, AMargin.Top,
@FCurrentExtent.a.Y, @FCurrentExtent.b.Y); FMargins.Bottom, FMargins.Top, @FCurrentExtent.a.Y, @FCurrentExtent.b.Y);
FScale.X := rX.CalcScale(1); FScale.X := rX.CalcScale(1);
FScale.Y := rY.CalcScale(-1); FScale.Y := rY.CalcScale(-1);
if Proportional then begin if Proportional then begin