this should belong to the editor.str_diag; 
i see no reason to inherit from DeleteAction; in fact DeleteAciton should be named RemoveFromDiagram, right?