Issue-Ref: 4191
Issue-Url: af3#4191

Signed-off-by: Andreas Bayha <bayha@fortiss.org>

Signed-off-by: Andreas Bayha <bayha@fortiss.org>

Add caught exception to list and throw new exception

See merge request !159

Issue-Ref: 4191
Issue-Url: af3#4191



Signed-off-by: Sebastian Bergemann <bergemann@fortiss.org>

Issue-Ref: 4168
Issue-Url: af3#4168



Signed-off-by: Tiziano Munaro <munaro@fortiss.org>

Issue-Ref: 4168
Issue-Url: af3#4168



Signed-off-by: Tiziano Munaro <munaro@fortiss.org>

Polish diagram appearance

See merge request !166

Issue-Ref: 4184
Issue-Url: af3#4184



Signed-off-by: Simon Barner <barner@fortiss.org>

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

When applying auto-layout inside a hierarchical component, the layout
would also affect the position of the component's port at the outer
level.

Example: File > Open AF3 Example > Load ACC Example, open component
"AdaptiveCruiseControl" and perform automatic layout inside it. Open the
containing component "ACC System". The input ports of
"AdaptiveCruiseControl" will have shifted.

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Work around a FX/SWT bug for scrolling on Linux

See merge request !165

Issue-Ref: 4183
Issue-Url: af3#4183



Signed-off-by: barner <barner@barner-VirtualBox>

Improve usability of diagram viewer

See merge request !163

Issue-Ref: 4181
Issue-Url: af3#4181



Signed-off-by: Simon Barner <barner@fortiss.org>

Issue-Ref: 4184
Issue-Url: af3#4184



Signed-off-by: Simon Barner <barner@fortiss.org>

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4181

Improve update performance of DynamicTreeTableViewer

See merge request !164

Issue-Ref: 4182
Issue-Url: af3#4182



Signed-off-by: Simon Barner <barner@fortiss.org>

Issue-Ref: 4181
Issue-Url: af3#4181



Signed-off-by: Simon Barner <barner@fortiss.org>

Issue-Ref: 4181
Issue-Url: af3#4181



Signed-off-by: Simon Barner <barner@fortiss.org>

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4181

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Uses a simple estimation of the extent of the name of a model
element based on the default JavaFX style. It does not use
refer to DiagramViewer, because the viewer delegates the
creation of content visuals to users and does not know the
font used in visuals. The visuals themselves are given bounds
and never compute the preferred size. A general solution would
be something like adding getPreferredSize to IVisual.

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

The coordinates of diagram ports were rounded to the next grid position,
while all other positions (bend points, component ports) were rounded
down. As a result, the position of diagram ports can sometimes be off
by one grid position. This probably was not noticed because the size
of diagram ports in the diagram tape measure was multiplied by 2, so
that the issue happend less often.

Example:
File > Open AF3 Example > Load ACC Example, open component
AdaptiveCruiseControl and do automatic layout. Some arrows that
should be horizontal are not quite.

Signed-off-by: Ulrich Schöpp <info@ulrichschoepp.de>
Issue-Ref: 4184

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4181

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

The position of aligned text depends on the width of the text,
which may change. This happens when a CSS stylesheet redefines
the default font. The diagram editor will open with slightly
misplaced text (e.g. on channels), which is only updated when one
moves the mouse. The class AlignedText updates the text
translation automatically when the text size changes.

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4183

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4184

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

For models with larger data dictionaries, the update of the data
dictionary can become slow enough that editing element names becomes
difficult. This commit changes the DynamicTreeTableViewer such that
not the whole tree is re-build on every update. This solves the
issue with the data dictionaries.

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4192

Ulrich Schöpp authored 3 years ago and Ulrich Schöpp committed 3 years ago

Zoom:
- When zooming with ctrl-scroll, the viewer fixes the content
  under the mouse pointer, so that zooming does not move the
  content.
- On Linux, zoom did not work proper. The grid was zoomed,
  but the content remained unchanged.

Panning:
- The viewer now supports panning with ctrl-drag.

Other changes:
- The code for zooming and moving the content has been simplified.
  Rather than applying transformations to all nodes separately and
  repeating coordinate calculations, the content scale and
  translation is now controlled by a single transformation.
  Coordinate transformations are done using JavaFX's methods.
- The GridCanvas has been optimized to avoid redrawing its content
  (which resulted in a noticable slow-down during panning). The
  grid canvas is not redrawn when the content is only moved.

Signed-off-by: Ulrich Schöpp <schoepp@fortiss.org>
Issue-Ref: 4181

4180: ModelContext.checkIDs(): Stop generation of error messages after 100 lines

Closes af3#4180

See merge request !162

4179 Add null check for model element handlers

See merge request !161

Issue-Ref: 4180
Issue-Url: af3#4180

Signed-off-by: Andreas Bayha <bayha@fortiss.org>