wmayer wrote: ↑Thu Dec 17, 2020 4:14 pm
Additionally git commit 627fea4e2 makes the spin box for the angle of the drill point wider again because with the current size the value is only partially visible.
Many thanks also to rynn.
There is only a minor issue: the angle is not aligned to the other spin boxes:
Attachments
FreeCAD_DoPmmAxNJs.png (13.49 KiB) Viewed 1748 times
There is only a minor issue: the angle is not aligned to the other spin boxes:
This is caused by the horizontal layout of the radio button and the spin box. Funny is that here the spin box is longer instead of shorter as the spin boxes above.
wmayer wrote: ↑Fri Dec 18, 2020 3:30 pmgit commit 9be9abe47
Yes, the trick is to move the spin box out of widget_2 that includes the two radio buttons.
EDIT:
rynn wrote: ↑Fri Dec 18, 2020 2:54 pm
This can be fixed too, but need changes to the dialog-structure.
Just saw your PR after I checked-in my modifications. I overwrote my changes with yours and pushed them to master.
uwestoehr wrote: ↑Fri Dec 18, 2020 4:28 pm
One of these commits introduced a regression:
- select a circle sketch and create a hole
- in the dialog select 'threaded'
result: you can no longer select if the thread is left- or right-handed because the radio group is disabled:
We only modified the .ui file but not the .cpp file, so we didn't change the program logic. The right/left option will be enabled when changing the threaded option three times. So there is an odd behaviour implemented in the .cpp file.
uwestoehr wrote: ↑Fri Dec 18, 2020 4:28 pm
One of these commits introduced a regression:
There is no regression at all but the new behaviour adds a case that is not handled correctly. In previous versions it was not possible to change the option "Threaded" if the profile is set to None. Now you can change this option even if profile is set to None and there an inconsistent initialization of the hole feature becomes noticeable.
The default value of Threaded is false which means that the ThreadDirection property is read-only. However, in the constructor this isn't set and thus the call of