document the code and start switching to event based instead of explicit attach methods
listening to 'show' events