We could, but this makes sense to me because there may be other places in the code where attachments are viewed or removed. Maybe not replaced. It just seems like good OO code org vs. overloading the event listener.

I'm not sure if the menu should close. I'll look at the wireframes again.