More helpful "need active body" message

[Meetlat]

Seems to me like a great enhancement!

[GeneFC]

I would definitely skip doing it if the response was overwhelmingly negative, or if there was a reason apart from "it wouldn't be that useful".

My only serious comment was that it did not appear to be useful for me. I am certainly not opposed unless it creates problems.

If others find it useful, great.

Gene

[jnxd]

Just an update on the progress on this thing. I've replaced the combo-view with a list and now most of the part design commands use the new dialog. Haven't done the better default selection, though. Additionally, the default action seems to be to create a new body if you just click OK without selecting anything. If anyone could help me out with this Qt related issue I'm having that'd be great.

[jnxd]

Ping

@chrisb, would you have some time to take a look at the needActiveBodyMessage() function and tell me how I could improve the logic to find the selected item?

[chrisb]

@chrisb, would you have some time to take a look at the needActiveBodyMessage() function and tell me how I could improve the logic to find the selected item?

I'm sad to say, but I'm the wrong person to answer this. I understand near to nothing from this code.

[jnxd]

@chrisb, would you have some time to take a look at the needActiveBodyMessage() function and tell me how I could improve the logic to find the selected item?

I'm sad to say, but I'm the wrong person to answer this. I understand near to nothing from this code.

Thanks for replying, Chris. I just pinged you because you were the most active person who's interested in this. Do you know anyone who could help?

[chrisb]

Can you help out?

pinged by pinger macro

pinged by pinger macro

[TheMarkster]

In a combobox you can use currentIndex() to find the currently selected item.

https://doc.qt.io/qt-5/qcombobox.html#currentIndex-prop

[TheMarkster]

To get the body that contains the active object currently selected:

Code: Select all

PartDesign::Body* bodyOfActiveObject = PartDesign::Body::findBodyOf(doc->getActiveObject());

[openBrain]

Can you help out?

TheMarkster already said it all (except I guess that 'activeObject' can be different from the selected item in some cases).
I quickly read the thread, and should say I would prefer a list rather than a combo box too. Especially if double-click is supported.