I suggest adding namespace to events in the setup, and here "offing" whole namespace - it's safer knowing that $overlay ($globalOverlay.find( '.ve-ui-window' )) might be used by some other extension possibly in the future. (In general things like that could be handled later on - but here I would make an exception because it have a possibility of being related in the future to bug that would be hard to debug.)