some ui improvements
use onclick listener object than implementing the onclick listener interface