modal now properly displays onclick