inline js added to buttons , will replace with external - time issue