ChartShape: Handle resize in a reasonable way
ClosedPublic

Authored by danders on Feb 2 2017, 10:57 AM.

Details

Summary

The chart shape can now be resized without having to resize and reposition
the different parts of the chart manually.

This patch:

  • Adds a cosmetic border to indicate to user the extent of the chart shape. (Only shown in words atm)
  • Introduces a ChartResizeStrategy class to the default tool that knows how to handle the contents of a chart shape.

Imo the ChartResizeStrategy should have been part of the chartshape
but this would have required BIC (and big) changes.
Adding it to the default tool makes it a small, non-disruptive change.
Making it a generic solution should imho wait for next major release.

Test Plan

Create a shart in words.
Resize it.
Do some undo/redos.
Move some parts (axis titles, legend around).
Resize the chart.
More undo/redos.
Apply share/rotation to chart
Resize.
More undo/redos.
Save/load
Repeat above exercises
Load chart from libreoffice
Repeate above exercises.

Diff Detail

Repository
R8 Calligra
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
danders updated this revision to Diff 10842.Feb 2 2017, 10:57 AM
danders retitled this revision from to ChartShape: Handle resize in a reasonable way.
danders updated this object.
danders edited the test plan for this revision. (Show Details)
danders added reviewers: boemann, mecir.
Restricted Application added a project: Calligra: 3.0. · View Herald TranscriptFeb 2 2017, 10:57 AM
This revision was automatically updated to reflect the committed changes.