DO NOT REVIEW 2378 Part Three: Return of the ROIFormWidget Class - Represent ROI: Export Tabs, Action Buttons, Table & Properties

[JackEAllen]

Issue Contributes to: #2378

Description

The proposed changes moves ROITableWidget, ROIPropertiesTableWidget, ROI export tabs and ROI action buttons into the class ROIFormWidget which will eventually act as the main interface for all these components when called within SpectrumViewerWindowView. This change is in preparation for extraction of the related xml GUI code out of spectrum_viewer.ui and into mantidimaging/gui/widgets/spectrum_widgets/ROIFORMWidget.ui and ROIFormWidget into mantidimaging/gui/widgets/spectrum_widgets/ROIFORMWidget.py

Developer Testing

• I have verified unit tests pass locally: python -m pytest -vs
• I have manually tested the Spectrum Viewer ROI table and properties remain up to date when:
  • Creating new ROIs
  • Renaming ROIs
  • Removing ROIs
  • Selecting an ROI with the intention to rename, then selecting another ROI
  • Swapping tabs between ROI and Image tabs
  • Toggling ROI visibility for various ROIs
  • Toggling ROI visibility and swapping between Image and ROI tab views.

Acceptance Criteria and Reviewer Testing

• Unit tests pass locally: python -m pytest -vs
• ROI Properties table values always display the selected ROI
• ROI table state remains up to date/in-sync with state of ROIs when adding, renaming and removing ROIs.
• The Selected ROI can only be a visible ROI
• No change in functional behaviour is introduced when compared to main branch.

Documentation and Additional Notes

Release notes are not needed as they will be added as part of the final PR completing the work described by #2378