moosetechnology / Moose

MOOSE - Platform for software and data analysis.
https://moosetechnology.github.io/moose-wiki
MIT License
136 stars 34 forks source link

Can't export as PNG with Roassal export button #1056

Closed seandenigris closed 9 years ago

seandenigris commented 9 years ago

Originally reported on Google Code with ID 1056

Describe the problem: what do you get? what do you expect?

When you add the export button, on a Roassal view and you select PNG export, you get:
MessageNotUnderstood: ROAthensImageExporter class>>subclassRespondibility

How to reproduce the problem: step by step if necessary

ROMondrianViewBuilder new exportButton ; open.

Clic on Export and Export as PNG.

Additional information: platform, context which may impact the problem

Moose 5.0 - W7

Please fill in the labels with the following information:
* Type-Defect
* Component-Roassal

Reported by v.blondeau92 on 2014-03-19 12:46:18

seandenigris commented 9 years ago
Fixed in the attached file.

There is another issue: 

when you export in PNG, the image exported contains only the visible objects in the
Roassal window. But the image size is like all the visualisation (with the hidden parts)
will be exported.

Reported by v.blondeau92 on 2014-03-19 12:52:30


seandenigris commented 9 years ago
And even when you change the size of the Roassal window, the exported image is always
of the same size...

The solution I propose is to export all the elements of the view, hidden or not.
See the attached fix.

However the tests ROMorphTest>>testBitmapSize and testImageForm have to be modified
to not fail.

Reported by v.blondeau92 on 2014-03-19 13:32:03


seandenigris commented 9 years ago
I integrated these changes now and it works. Thanks.

As you said, the two tests are failing. Can you provide correct versions of these tests?

Reported by tudor@tudorgirba.com on 2014-07-10 04:59:21

seandenigris commented 9 years ago
Yes! The tests are attached

Reported by v.blondeau92 on 2014-07-15 09:03:51


seandenigris commented 9 years ago
Thanks! I integrated them now.

Reported by tudor@tudorgirba.com on 2014-07-23 20:58:52

seandenigris commented 9 years ago

Reported by tudor@tudorgirba.com on 2014-07-24 05:15:37

seandenigris commented 9 years ago
Issue 995 has been merged into this issue.

Reported by tudor@tudorgirba.com on 2014-07-24 05:16:01