osate / osate2

Open Source AADL2 Tool Environment
http://osate.org
Eclipse Public License 2.0
36 stars 8 forks source link

GE SVG Export isn't editable / high quality #1461

Closed sprocter closed 3 years ago

sprocter commented 6 years ago

Summary

I have been using the graphical editor to make screenshots for a paper I'm writing, and SVG is ideal because it prints very clearly. Exported diagrams, however, do not render cleanly. Text labels, in particular, are blurry:

screenshot from 2018-08-17 11-30-53

Expected and Current Behavior

Expected: Text should be clearly readable, and fully scalable. Also ideal would be that the individual shapes are editable in an SVG editor like inkscape

Actual: Text is blurry. The entire image is rendered as one object, and so is not editable.

Steps to Reproduce

  1. Create an AADL model
  2. Export an SVG version of the graphical view via right-clicking on the diagram and selecting "Export Diagram..."

Environment

philip-alldredge commented 6 years ago

Can you provide an SVG that is generated from your development environment? This behavior does not seem consistent with the behavior under windows.

sprocter commented 6 years ago

Here's the .svg file: https://gist.github.com/sprocter/e99ff8f3308942f941f8a86802bb1f84

philip-alldredge commented 6 years ago

The graphic is editable. In inkscape, it is possible to ungroup the object and move individual pieces. This seems to be consistent between platforms.

The font behavior differs between platform. The font-family differs between platforms. Also on windows, the text is exported as SVG text and is accordingly editable. On Ubuntu the text appears to be exported as an embedded PNG image. This causes a large SVG file and low quality.

To resolve, we'll need to dig into the behavior of the SVG exporter to determine the root cause. The graphical editor relies on Graphiti. Graphiti's SVG export capability uses Batik's SVGGraphics2D.

philip-alldredge commented 6 years ago

If a migration to GEF5 is performed, then the SVG export could be reworked as part of that.

philip-alldredge commented 3 years ago

Fixed by dbefc8b