Length, depth and x angle properties now update when the field is updated

[Gorgious56]

I find it mildly annoying to have to click the Refresh button next to the properties for the object geometry to be updated.

This PR removes these buttons and adds a callbacks to the properties so the geometry is updated every time the fields are updated.

Pros : realtime feedback

Cons : may lag user computer when scrubing the fields with a lot of items selected. I think this is not really an issue since the purpose of these fields is to input a precise value so I think it's unlikely the users would scrub them in the case they have multiple elements selected. It's not that bad either for simple shapes.

I also set the minimal values to 0 since negative lengths don't really make sense (unless they do in the ifc schema ?) and the geometry creator doesn't seem to like it either.

[Moult]

From memory I believe the reason this isn't the case is because it actually edits IFC data. I have to check whether or not these edit in-place or if they create and delete in the IFC (which would then start incrementing the step ID unnaturally).

[Gorgious56]

I also contemplated displaying the ° for x_angle with subtype="ANGLE", to convey they're degrees and not radians, and it might make the subsequent calculations more straightforward.

Since after looking it up it seems this property is systematically converted to radians afterwards in the code.
eg

https://github.com/IfcOpenShell/IfcOpenShell/blob/v0.7.0/src/blenderbim/blenderbim/bim/module/model/wall.py#L251

So to be clear the displayed value would be in degrees, but the actual sotred value would be in radians.

It would mean changing a bit more than a few lines to accomodate for the change, though.

[Gorgious56]

From memory I believe the reason this isn't the case is because it actually edits IFC data. I have to check whether or not these edit in-place or if they create and delete in the IFC (which would then start incrementing the step ID unnaturally).

Oh right then this makes sense. A bit unfortunate, but understandable for the sake of having a sane bookkeeping.

[Gorgious56]

@Moult should I close this PR ? I can check the IDs increment problem if you think that's the only thing preventing adoption.

[Moult]

Yes I suspect many elements will be created / removed in IFC, and this may also have an impact on the IFC undo step. I haven't had time yet (rushing for this upcoming release) to check these suspicions.

[Andrej730]

Just noticed that it's already kind of possible with redo popup, not sure about the consequences though...

blender_6ickDWFkTq.mp4