Closed jwiggins closed 11 years ago
Interesting - I'd assumed that you'd implemented this on the Python/Traits side rather than the Enaml side. This is pretty clean, though.
I'm not aware of a way to hook up notifications via the Python interface. Do you know a way?
Okie dokie. Now that I've added the DefaultEditor
I'm pretty happy with this.
In terms of the notifications, I hadn't thought hard about it, but we don't need to mimic the full notification operator system, rather you just need to know how to sync up a Trait with an Atom on the underlying object (basically hook up appropriate handlers for each system which push changes with appropriate care to avoid loopback).
A 50-50 option would be to have the editors written in Enaml, using the Enaml notification to handle synchronisation, but generate the Windows, Containers, Forms, Labels, etc programatically from Python, rather than using a looper.
In any case, that's not needed if things work this way just fine.
I believe that covers everything now.
BTW, I'm tempted to merge this in the next day or so, since it's a needed feature and I'll take that over something perfect any day.
Please do. Easy extensibility can be covered by a future PR.
This is still incomplete, but I think it shows what's possible.
(I also fixed setup.py since it had the wrong URL for the project)