| Hi,
Tom is right concerning the SWT/JFace dependency as long as Zest.FX is concerned, which should be targeted at providing JavaFX visualization also in a standalone scenario (and should thus have no Eclipse UI dependencies).
However, as Zest.FX.UI is intended to cover the Eclipse UI integration of Zest.FX (i.e. embed the Zest.FX viewer into an Eclipse View, additionally also integrating it with the properties view, etc.), it would depend on SWT and the Eclipse UI anyway.
The question that IMHO arises is whether an API that provides the underlying data structures to the Zest.FX viewer (i.e. what is currently covered by the JFace API in Zest) should only be available in the Zest.FX.UI context (which would be the implication if we use JFace content and label providers) or also in the standalone scenario.
If we assume that we split the GEF4 Graph into an interface and implementation part, could we not simply use the GEF4 Graph interfaces (or a lightweight facade built on top of it) for exactly this purpose?
Cheers Alexander Hi,
Yeah the paradigm is ok and I have already thought about different APIs
multiple times (even implemented for customers).
I'm in the process to define such an API as part of e(fx)clipse although
there's no source yet available in the repo. Once I have something I'll
post an update so that you can get an idea ;-)
Tom
On 08.07.14 11:19, Ujhelyi Zoltán wrote:
You are right, the hard SWT dependency is the killer here. :) The JFace API itself is not good...
I will do some research about alternative patterns already used (I am not confortable with designing an entirely new paradigm here ;) ).
Thanks for the feedback,
Zoltán
-- Zoltán Ujhelyi
https://www.inf.mit.bme.hu/en/members/ujhelyiz
Fault Tolerant Systems Research Group
Budapest University of Technology and Economics
On 2014.07.08., at 11:13, Tom Schindl <tom.schindl@xxxxxxxxxxxxxxx> wrote:
[...]
In summary, I guess, JFace Viewers API support is still not a strict requirement, but I would have troubles designing a different API that is still at least as good for creating the displayable models.
JFace would drag in SWT and isn't GEF4 to be widget toolkit neutral?
Beside that I think the JFace-API shows is age in it's API:
* useage of Array-Types (=> you can not generify it completely no
matter how many GOSC projects you setup)
* No SAM-Types wouldn't you want to write a label-provider like this in
Java8
Viewer<Person> v = new Viewer<>();
v.setTextLabelProvider( (o) -> o.name );
* Hard SWT-Dependency
* (I could go on with more :-)
Tom
_______________________________________________
gef-dev mailing list
gef-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/gef-dev
_______________________________________________
gef-dev mailing list
gef-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/gef-dev
_______________________________________________
gef-dev mailing list
gef-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/gef-dev
--
Dr. Alexander Nyßen
Dipl.-Inform.
Software-Engineer
Telefon: +49 (0) 231 / 98 60-210
Telefax: +49 (0) 231 / 98 60-211
Mobil: +49 (0) 151 / 17396743
http://www.itemis.de
alexander.nyssen@xxxxxxxxx
itemis AG
Am Brambusch 15-24
44536 Lünen
Rechtlicher Hinweis:
Amtsgericht Dortmund, HRB 20621
Vorstand: Jens Wagener (Vors.), Wolfgang Neuhaus, Dr. Georg Pietrek, Jens Trompeter, Sebastian Neus
Aufsichtsrat: Dr. Burkhard Igel (Vors.), Stephan Grollmann, Michael Neuhaus
--
Dr. Alexander Nyßen
Dipl.-Inform.
Software-Engineer
Telefon: +49 (0) 231 / 98 60-210
Telefax: +49 (0) 231 / 98 60-211
Mobil: +49 (0) 151 / 17396743
http://www.itemis.de
alexander.nyssen@xxxxxxxxx
itemis AG
Am Brambusch 15-24
44536 Lünen
Rechtlicher Hinweis:
Amtsgericht Dortmund, HRB 20621
Vorstand: Jens Wagener (Vors.), Wolfgang Neuhaus, Dr. Georg Pietrek, Jens Trompeter, Sebastian Neus
Aufsichtsrat: Dr. Burkhard Igel (Vors.), Stephan Grollmann, Michael Neuhaus
|
