[Mark Sir Link]

you should probably reserve functions with on prefixed for actual events that are being thrown.

also rather than constantly creating the UI each time you want to use it, create the UI controls the first time and hide/show as necessary.