Skip to content

Commit 57c10fd

Browse files
committed
make CutGeometry's flip parameter and Invert checkbox agree
1 parent 4a38dc2 commit 57c10fd

2 files changed

Lines changed: 11 additions & 6 deletions

File tree

src/OpenCOVER/plugins/general/CuttingSurface/CuttingSurfaceInteraction.cpp

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,8 @@ CuttingSurfaceInteraction::getParameters()
209209
int numoptions;
210210
char **labels;
211211
inter_->getChoiceParam(OPTION, numoptions, labels, option_);
212+
213+
inter_->getBooleanParam("flip", flip_);
212214
}
213215

214216
void CuttingSurfaceInteraction::createMenu(bool invertClip)
@@ -221,15 +223,11 @@ void CuttingSurfaceInteraction::createMenu(bool invertClip)
221223
clipInvertCheckbox_->setCallback(
222224
[this](bool state)
223225
{
226+
plugin->getSyncInteractors(inter_);
224227
plugin->setBooleanParam("flip", state);
225228
plugin->executeModule();
226-
227-
if (clipPlaneFlipCheckbox_)
228-
{
229-
bool flip = clipPlaneFlipCheckbox_->state();
230-
clipPlaneFlipCheckbox_->setState(!flip);
231-
}
232229
});
230+
clipInvertCheckbox_->setState(flip_ != 0);
233231
}
234232

235233
optionChoice_ = new ui::SelectionList(menu_, "SurfaceStyle");
@@ -285,6 +283,7 @@ void CuttingSurfaceInteraction::createMenu(bool invertClip)
285283
break;
286284
}
287285
}
286+
plugin->getSyncInteractors(inter_);
288287
plugin->executeModule();
289288
}
290289
});
@@ -385,6 +384,11 @@ CuttingSurfaceInteraction::updateMenu()
385384
break;
386385
}
387386

387+
if (clipInvertCheckbox_)
388+
{
389+
clipInvertCheckbox_->setState(flip_ != 0);
390+
}
391+
388392
updatePickInteractors(showPickInteractor_);
389393
updateDirectInteractors(showDirectInteractor_);
390394
}

src/OpenCOVER/plugins/general/CuttingSurface/CuttingSurfaceInteraction.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ class CuttingSurfaceInteraction : public ModuleInteraction
9797
NumSurfaceStyles
9898
};
9999
int option_ = OPTION_NONE;
100+
int flip_ = 0;
100101
ui::SelectionList *optionChoice_=nullptr;
101102

102103
CuttingSurfacePlane *csPlane_;

0 commit comments

Comments
 (0)