feat(annotations): Toggle region annotation mode

[mxiao6]

No description provided.

[ConradJChan]

I wonder if the switch is needed at all. Should it just blindly run through all the update*Button methods each time? For instance if the click region and then click highlight, we would need region and highlight to update respectively

[mxiao6]

I wonder if the switch is needed at all. Should it just blindly run through all the update*Button methods each time? For instance if the click region and then click highlight, we would need region and highlight to update respectively

This is deactivateCurrentControl(), which only deactivate the current control. Update next control will happen in handle*Click.

[ConradJChan]

The call to this.deactivateCurrentControl() seems unnecessary. It seems like this method's responsibility should be (and all handle([Region|Highlight|Draw])Click for that matter):

• update currentActiveControl
• update all buttons states accordingly
• callback to Viewer with the type toggled

so maybe something like this

this.currentActiveControl = this.currentActiveControl === 'region' ? 'none' : 'region';
this.controlsMap.keys().forEach(updateButton => updateButton());
onModeClick('region');

Thoughts?

[ConradJChan]

Update method description?

[mxiao6]

Update method description?

I think it's the same meaning 😂