Improve tooltip for tools Rotation, Flipping by line and Flipping by axis.

The app will show updated tooltip message in case a user selected point that is
already a part of the list of operated objects.

--HG--
branch : develop
This commit is contained in:
Roman Telezhynskyi 2016-09-17 19:16:15 +03:00
parent 33298c3361
commit 255db32720
3 changed files with 14 additions and 5 deletions

View file

@ -192,6 +192,7 @@ void DialogFlippingByAxis::ChosenObject(quint32 id, const SceneObject &type)
{ {
if (objects.contains(id)) if (objects.contains(id))
{ {
emit ToolTip("Select origin point that is not part of the list of objects");
return; return;
} }

View file

@ -189,14 +189,15 @@ void DialogFlippingByLine::ChosenObject(quint32 id, const SceneObject &type)
{ {
if (type == SceneObject::Point) if (type == SceneObject::Point)
{ {
if (objects.contains(id))
{
return;
}
switch (number) switch (number)
{ {
case 0: case 0:
if (objects.contains(id))
{
emit ToolTip("Select first line point that is not part of the list of objects");
return;
}
if (SetObject(id, ui->comboBoxFirstLinePoint, tr("Select second line point"))) if (SetObject(id, ui->comboBoxFirstLinePoint, tr("Select second line point")))
{ {
number++; number++;
@ -207,6 +208,12 @@ void DialogFlippingByLine::ChosenObject(quint32 id, const SceneObject &type)
} }
break; break;
case 1: case 1:
if (objects.contains(id))
{
emit ToolTip("Select second line point that is not part of the list of objects");
return;
}
if (getCurrentObjectId(ui->comboBoxFirstLinePoint) != id) if (getCurrentObjectId(ui->comboBoxFirstLinePoint) != id)
{ {
if (SetObject(id, ui->comboBoxSecondLinePoint, "")) if (SetObject(id, ui->comboBoxSecondLinePoint, ""))

View file

@ -234,6 +234,7 @@ void DialogRotation::ChosenObject(quint32 id, const SceneObject &type)
{ {
if (objects.contains(id)) if (objects.contains(id))
{ {
emit ToolTip("Select origin point that is not part of the list of objects");
return; return;
} }