Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 419467

Summary: Allow layout customization for SpaceTree
Product: [Tools] GEF Reporter: Istvan Rath <istvanrath>
Component: GEF ZestAssignee: gef-inbox <gef-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: nyssen, zoltan.ujhelyi
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch for SpaceTreeLayoutAlgorithm nyssen: iplog+

Description Istvan Rath CLA 2013-10-15 10:24:03 EDT 
I'm proposing a small patch to allow for inherited classes to customize certain aspects of the SpaceTree layout.
Comment 1 Istvan Rath CLA 2013-10-15 10:26:16 EDT 
Created attachment 236487 [details]
Patch for SpaceTreeLayoutAlgorithm
Comment 2 Zoltan Ujhelyi CLA 2013-10-15 18:18:10 EDT 
I have added the code to master with the following changes:
 * Made getters/setters public to be more consistent with other configurable layout algorithms.
 * Added some Javadoc comments.
Comment 3 Alexander Nyßen CLA 2014-05-22 01:57:29 EDT 
Comment on attachment 236487 [details]
Patch for SpaceTreeLayoutAlgorithm

As the patch seems to have been applied (with modification), adding iplog flag.
Comment 4 Alexander Nyßen CLA 2014-05-22 02:05:29 EDT 
Setting assignee to Zoltan and changing component to GEF4 Zest, as this affected the Zest2 code base.
Comment 5 Alexander Nyßen CLA 2015-03-18 07:22:58 EDT 
(In reply to Istvan Rath from comment #1)
> Created attachment 236487 [details]
> Patch for SpaceTreeLayoutAlgorithm

Istvan, could you please confirm that your contribution complies with http://www.eclipse.org/legal/CoO.php.
Comment 6 Istvan Rath CLA 2015-03-18 07:27:52 EDT 
(In reply to Alexander Nyßen from comment #5)
> (In reply to Istvan Rath from comment #1)
> > Created attachment 236487 [details]
> > Patch for SpaceTreeLayoutAlgorithm
> 
> Istvan, could you please confirm that your contribution complies with
> http://www.eclipse.org/legal/CoO.php.

I hereby confirm that my patch is in full compliance with http://www.eclipse.org/legal/CoO.php
Comment 7 Alexander Nyßen CLA 2015-03-18 16:48:58 EDT 
(In reply to Istvan Rath from comment #6)
> (In reply to Alexander Nyßen from comment #5)
> > (In reply to Istvan Rath from comment #1)
> > > Created attachment 236487 [details]
> > > Patch for SpaceTreeLayoutAlgorithm
> > 
> > Istvan, could you please confirm that your contribution complies with
> > http://www.eclipse.org/legal/CoO.php.
> 
> I hereby confirm that my patch is in full compliance with
> http://www.eclipse.org/legal/CoO.php

Thank you Istvan!