My preference would be for the controller to deal with UI events. This doesn't seem to do anything general. What do you think?