This is an interesting question. I refactored this code yesterday to make XBlockView the parent of ModuleEditView, and I wasn't entirely happy with this either. Really I should introduce a new subclass for the container view to use, but I wasn't sure what to call it. It seemed that eventually we just want XBlockView, and ModuleEditView will be discarded.