Colour Panel Issue

[Owen Arnold]

See issue here ​https://github.com/mantidproject/documents/blob/master/Design/VATES/VSI_PV431_Integration_Issues.md

We need a ParaView fix for this, but assigned to a Mantid developer to track it.

[Steven Hahn]

Ref: ​https://github.com/mantidproject/documents/blob/master/Design/VATES/VSI_PV431_Integration_Issues.md#colour-editor-panel

Issue: Colour Editor Panel

Diagnosis:

MdViewerWidget instance created in MantidUI::showVatesSimpleInterface() is not destroyed when the window is closed. To confirm, I simply put an abort() in destructor for MdViewWidget. That doesnt get called when I close the VSI window. (I think is doesn't get called when with Mantid exits). However, every time the showVatesSimpleInterface() is called, it does indeed create a new instance of MdViewerWidget. We need to either use the same MdViewerWidget or make sure it's destroyed when the window is closed.

With MdViewerWidget not getting destroyed, it doesn't cleanup the old COLOR_EDITOR_PANEL registered with the pqApplicationCore and hence the error message.

Solution: I am assuming you will take care of figuring out how to ensure that MdViewerWidget gets deleted on your side since I am not too familiar with that code. Let me know if you want me to take a stab, and I surely can.

Utkarsh

[Steven Hahn]

Integration issues were fixed in ticket #7363.

[Stuart Campbell]

This ticket has been transferred to github issue ​12359