Fix copy/paste bugs in reflectometry GUI

Fix bugs with row highlighting in reflectometry GUI

Remove unused file in restructured GUI code

Fix help button segfault on reflectometry GUI

Change shortcut to avoid clash on on workbench

I think this was needed in the original restructuring but the design has been simplified and it is no longer necessary

Re #26025

- This was to do with the way the presenter is created as an optional. It doesn't need to be an optional here - a unique pointer is better and makes ownership clearer.
- This commit also includes some tidying of the MainWindowSubscriber which was being used incorrectly.
- It also includes replacing some calls to m_presenter with calls to m_notifyee. They are the same object but we should use m_notifyee to be more in keeping with MVP. There is one remaining use of m_presenter in closeEvent but it is beyond the scope of this PR to fix that.

Re #23027

Fix ICat Login for new Reflectometry GUI on Workbench

Re #23027

Adding ads observer to muon analysis

This avoids a clash on workbench

Re #23027

Reflectometry GUI: Change defaults for Instrument Settings

Re #23027

make GetEiT0atSNS to work with monitor data with TOF not wrapped to first frame

Port Muon ALC Interface

Fix adding rows bug when erasing rows

Fix autosaving of group workspaces on the reflectometry GUI

Expose indexOf to python

Update SANS test script

Fast plotting of MatrixWorkspaces in SliceViewer

This is a duplicate of a mock that's in ReflMockObjects. Longer term I'd
like to separate it out into its own file but for now it's confusing so
remove the unused version.

Re #23027

Re #23027

Add unit test to catch bug

re #25930

Move export table check to its own section

This removes ambiguity about what export table should save

re #26002

Remove previous fix for updating state on batch complete and instead do
it on autoreduction resumed. This is when the clearing of state is
happening so fix it at this point.

Re #23027