jany_ has uploaded a patch.
Description:
I stumbled upon the following things:
- If the Pan-button is clicked and then clicked again, the button is no longer checked but the function still works.
- The Dotspatial.Controls.MenubarHeader.UncheckButtonsExcept functions runs through every ToolStripButton and unchecks it, even though there should be only one ToolStripButton checked.
- It seems as if the Dotspatial.Controls.SimpleActionItem.Toggling function is never used.
I changed the code so that the currently selected ToolStripButton is remembered and only unchecked if it’s clicked again. Then the FunctionMode gets reset to None. If another ToggleButton is clicked the currently checked ToggleButton gets unchecked and the new one is remembered.
I removed the Dotspatial.Controls.SimpleActionItem.Toggling and added OnUncheck because I have some code that reacts only if the currently activated ToolstripButton becomes unchecked.
The Dotspatial.Controls.SimpleActionItem.OnClick is only run, if the ToolStripButton gets checked, because it doesn’t make sense to run if it’s unchecked.