missing feature to change loft section order

[uwestoehr]

Hmm, after you merged my https://github.com/FreeCAD/FreeCAD/pull/3876 and your subsequent fixes, the intended feature does no longer work:
I have a sweep with several sections and I cannot select one section and press DEL. I can use the context menu to delete, but not the DEL key.

I cannot see from the code why this happens. I worked yesterday on my development PC.

[chrisb]

However, in your example, the resulting part's outer wall is black (the color I get when e.g. unions fail):

That means only that the inside is out, which happens because the shape starts inside in the other direction.

[wmayer]

What about my PR for lofts:
https://github.com/FreeCAD/FreeCAD/pull/3871
Do I have to change there anything to be merged?

I can only merge one PR after the other...

Hmm, after you merged my https://github.com/FreeCAD/FreeCAD/pull/3876 and your subsequent fixes, the intended feature does no longer work:
I have a sweep with several sections and I cannot select one section and press DEL. I can use the context menu to delete, but not the DEL key.

I haven't touched anything in this regard. Are you sure it has worked before? When I use the Del key in the sections panel I get this Qt warning:

QAction::eventFilter: Ambiguous shortcut overload: Entf

Since you have added the Del shortcut to all three sub-panels and they are all active at the same time Qt is not able to handle this. A workaround is to hide the other two sub-panels with the arrow buttons.

[uwestoehr]

I can only merge one PR after the other...

You misunderstood me. I know your spare time is limited. I was just seeing merge conflicts because of your recent changes. So I wondered if I have to do something according to your recent changes. So no time pressure but need to learn. (You know I will have to extend the pad feature soon and see that I need to learn more in the UI field.)

I haven't touched anything in this regard. Are you sure it has worked before?

Argh, I am so untalented. To keep the overview I collapsed the other 2 lists and only worked on one. And yes, with my test setup:

FreeCAD_kCEFDhIuAh.png (15.4 KiB) Viewed 848 times

it works.

Of course I cannot bind the same key to 3 different lists. Stupid me.

[wmayer]

You misunderstood me. I know your spare time is limited. I was just seeing merge conflicts because of your recent changes. So I wondered if I have to do something according to your recent changes. So no time pressure but need to learn. (You know I will have to extend the pad feature soon and see that I need to learn more in the UI field.)

Ah, no action required. I will merge it soon and fix the conflicts myself.

Argh, I am so untalented. To keep the overview I collapsed the other 2 lists and only worked on one. And yes, with my test setup:

Don't worry it's not about talent, it's about experience and to learn the pitfalls of the used libraries. So, far you did a very good job with all your PRs.

[uwestoehr]

...So I wondered if I have to do something according to your recent changes...

There was one according to your yesterday's commits. Here's the PR: https://github.com/FreeCAD/FreeCAD/pull/3877