Leader Line issues

[uwestoehr]

Ahh, you've run into the great FC divide between App and Gui! Gui is allowed to call App, but App is not allowed to call Gui.

OK. The idea was only to add the properties LineWidth etc. if the parameter ShowFrame is set.
So if it is possible to evaluate ShowFrame in ViewProviderRichAnno.cpp then my idea can work, otherwise i don't see a ways to implement a conditional hiding of the line parameters when there is no frame.

[wandererfan]

I don't know about the internals. What i did was just what i see for other TD components and they do a QGIView updateView(true).

Usually Qt figures out all the repainting for you. In this case it was not, so we need to force it with the update() call.

[wandererfan]

OK. The idea was only to add the properties LineWidth etc. if the parameter ShowFrame is set.
So if it is possible to evaluate ShowFrame in ViewProviderRichAnno.cpp then my idea can work, otherwise i don't see a ways to implement a conditional hiding of the line parameters when there is no frame.

Code: Select all

if (getViewObject()->ShowFrame.getValue()) {
       LineWidth.setStatus(App::Property::ReadOnly,true);    //or
//       LineWidth.setStatus(App::Property::Hidden,true);
       ....
}

[wandererfan]

OK. The idea was only to add the properties LineWidth etc. if the parameter ShowFrame is set.
So if it is possible to evaluate ShowFrame in ViewProviderRichAnno.cpp then my idea can work, otherwise i don't see a ways to implement a conditional hiding of the line parameters when there is no frame.

Code: Select all

if (!getViewObject()->ShowFrame.getValue()) {
       LineWidth.setStatus(App::Property::ReadOnly,true);    //or
//       LineWidth.setStatus(App::Property::Hidden,true);
       ....
}

[uwestoehr]

Code: Select all

if (getViewObject()->ShowFrame.getValue()) {
       LineWidth.setStatus(App::Property::ReadOnly,true);    //or

Thank you. I tried this but chose the wrong position directly after the addition of the parameters.
In this PR: https://github.com/FreeCAD/FreeCAD/pull/2956
it works now as it should: only enabled if there is a frame.