Gui: improve SelectionSingleton::setVisible()

Change argument to enum type. Make setVisible() robust against
unexpected selection change.
This commit is contained in:
Zheng, Lei
2019-09-05 11:28:53 +08:00
committed by wmayer
parent f57283f598
commit 1e69eb0d8e
3 changed files with 70 additions and 19 deletions

View File

@@ -793,7 +793,7 @@ StdCmdToggleVisibility::StdCmdToggleVisibility()
void StdCmdToggleVisibility::activated(int iMsg)
{
Q_UNUSED(iMsg);
Selection().setVisible(-1);
Selection().setVisible(SelectionSingleton::VisToggle);
}
bool StdCmdToggleVisibility::isActive(void)
@@ -867,7 +867,7 @@ StdCmdShowSelection::StdCmdShowSelection()
void StdCmdShowSelection::activated(int iMsg)
{
Q_UNUSED(iMsg);
Selection().setVisible(true);
Selection().setVisible(SelectionSingleton::VisShow);
}
bool StdCmdShowSelection::isActive(void)
@@ -894,7 +894,7 @@ StdCmdHideSelection::StdCmdHideSelection()
void StdCmdHideSelection::activated(int iMsg)
{
Q_UNUSED(iMsg);
Selection().setVisible(false);
Selection().setVisible(SelectionSingleton::VisHide);
}
bool StdCmdHideSelection::isActive(void)