Cleanup-AbstractTool #523
Cleanup-AbstractTool #523
Conversation
…ion (which is a horrible thing, too). But this way we have one strange things removed (AbstractTool) - rewrite users of lookupVar:declare: -
|
I read the changes and it is unclear to me why self model is worse than SystemNavigation new. |
|
the model then was just forwarding to SystemNavigation for these methods AbstractTool is just used by old tools.. e.g. the inspector or debugger do no use it. Just ChangeSorter (it's model was a subclass), MessageList (which was just doing AbstractTool new) and the too for Manifest (it's model was a subclass of AbstractTool). We need something, but just one abstraction. This step removes AbstractTool, as the code was strange and it was mostly just forwarding to SystemNavigation. We then need to continue, e.g SystemNavigation with all the UI related methods is very strange. |
Rewrite all users of AbstractTool, move some feature to SystemNavigation (which is a horrible thing, too).
But this way we have one strange things removed (AbstractTool)