swcarpentry / windows-installer

Software Carpentry installer for Windows.
MIT License
21 stars 17 forks source link

Prevent checkbox that runs main install script from being unchecked #29

Closed ethanwhite closed 9 years ago

ethanwhite commented 9 years ago

Fixes #28

ethanwhite commented 9 years ago

There is an option to hide the checkbox entirely, but I thought that might prove more confusing since something will run (visibly) after clicking Finish.

wking commented 9 years ago

I have no Windows machine to test it on, but this approach looks reasonable to me.

ethanwhite commented 9 years ago

@kbroman - if this sounds like it will solve your issue in #28 I'll go ahead and get it merged.

kbroman commented 9 years ago

This, looks great, thanks!

ethanwhite commented 9 years ago

Thanks for reporting this @kbroman. I'll get it on the To Do list to roll another release so that we can get this rolled out for upcoming workshops.