Don't create OperationCodeEditor instance from the outputViewer panel when not in debug mode

[umesh-timalsina]

While working on #1800 , I found that all other TextEditorWidget sub-classes on the prototypical inheritance chain are created only once when the particular visualizer is selected. However, the UI creates multiple (3) instances of LogViewerWidget when loading. This will be a problem since monaco relies on DisposableStore which requires widgets to be properly disposed/destroyed. Regardless of monaco, I think this needs some investigation on why multiple widgets are instantiated.

[umesh-timalsina]

Branch: 571-monaco-text-editor Branch: master

[umesh-timalsina]

I found that this error stems from TilingVizPanel in which upon selection of LogViewerWidget to view the output OperationCodeEditor panel is also selected (Which should not be the case because the execution was not run in debug mode) and the problem with disposable entries goes away when using monaco-editor from #1800 for any operation in debug mode.

[brollb]

Hmm... When I added logs for text editors being destroyed as well, it looks like the first two should be deleted correctly: