why you're namespacing the events? we are not sharing the `$viewer` element with anyone and we don't expect someone to attach handlers in another namespace ... you should only namespace events when you need to and when you'll be using `.off` somewhere. Since the $viewer is removed automatically by brackets, we don't have to explicitly call `.off` here and so we don't need `.HistoryViewer` namespace for events