- Pulling out variable selection instantiation logic into a file called choose-settings which is then pulled in by chooseSettings.html.
- Removed inline javascript initialisation in the dialog that was registering the add and close behaviour when submitting chosen variables
- Moved variable selection handler into a variable-selection file

Issue: GCP-9122
Reviewer: Rebecca B
Fixing up jshint errors in the variable submission method in variable-selection.js, and moving over a couple of methods from nurseryManager.js that aren't used anywhere else.

Issue: GCP-9122
Reviewer: Rebecca B
Changed to an event driven communication between the variable selection dialog and the choose settings screen.

Issue: GCP-9122
Reviewer: Rebecca B
Changing the addSettings method to accept JSON. Hard coding a JSON response to emulate what I will be sending back in the future when variables are selected.

Issue: GCP-9122
Reviewer: Rebecca B
Removing no longer used functions.

Issue: GCP-9122
Reviewer: none
Changing the way I instantiate the the Variable Selection dialog to instantiate once and show on demand with different data.

Issue: GCP-9122
Reviewer: Rebecca B
Tidying the chooseSettings html a little. Stripping down the modal to the bare minimum (just a test 'add variable' button and the done button).

Issue: GCP-9122
Reviewer: Rebecca B
Removing test data and hooking up to real services. Introducing use of handlebars for templating.

Issue: GCP-9122
Reviewer: Rebecca B
Removing choose-settings.js, which was made redundant by my changes.

Issue: GCP-9122
Reviewer: none
Updating URL to reflect changes to the new variable usage .

Issue: GCP-9122
Reviewer: Rebecca B
Removing popular variables until we have properly implemented the service.

Issue: GCP-9122
Reviewer: Rebecca B
Scoping event handler so as to ensure it gets destroyed when the page reloads after a save (making sure event handlers don't end up getting registered twice).

Issue: GCP-9122
Reviewer: Rebecca B (post commit)