I put the GEF editor back into the Editor module. Why? Because in order to do Undo/Redo I have to use GEF’s Command Stack and use its Commands in the Tree View and so the Editor module now has a dependency on GEF. In fact it has a dependency on the Diagram Editor. So why not put them all in one module?