Refs #9170 Restructured Reflectometry Interface

The interface now follows the same pattern as the ORNL SANS interface for declaration. The ReflGui class is now the window itself, this means it recieves the close signal from the mantid main window and can override some things within itself.

Tidied up the connections, as there was no need for most of the handlers.

Renamed most of the methods to they follow python's style more closely.

Added more whitespace
Refs #9170 fixed a typo and added a rethrow

An except wan't rethrowing when it should have, so i made it rethrow

there was a typo on one of the renamed methods
Refs #9170 Merge master in to add other recent features

Merges changes from #9039 and #9043, tehy still need refactored for the new gui style though.

Merge remote-tracking branch 'origin/master' into bugfix/9170_Refl_stopped_working_error

Conflicts:
	Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py
Refs #9170 More refactoring and tidying

The merged changes have been updated to be consistant in terms of connections and naming conventions

White space has been imporved

Existing docstrings have been updated to double quotes rather than single and have been moved to the proper place below Def. Some docstrings have been changed to comments

Some initialisations have been moved to the constructor
Refs #9170 Added Docstrings to all methods

All methods now have a docstring in the right place
Refs #9170 Factored out numbers and renamed variables

There were a lot of column indexes used as constants, they've been factored out into member variables

A few varaibles have been renamed as they didn't follow pyhton style
Refs #9170 Fixed typos and missed connection

The choose columns button on the interface missing a connection

There were a couple of variables with typos after the refactor.
refs #9170 merge conflict with develop

Merge branch 'bugfix/9170_Refl_stopped_working_error' into develop

Conflicts:
	Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py