bpy.types.Gizmo's boolean scaling properties seem incorrectly inter-related #73858

New Issue

Jesse Yurkovich · 2020-02-15T23:03:12+01:00

Jesse Yurkovich commented

2020-02-15 23:03:12 +01:00

System Information
Operating system: Windows-10-10.0.18362-SP0 64 Bits
Graphics card: GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 441.66

Blender Version
Broken: version: 2.83 (sub 3), branch: master, commit date: 2020-02-14 23:40, hash: 92a56bbe6a
Worked: (optional)

Short description of error

The gist is that:

mpr = self.gizmos.new(...)
mpr.use_draw_scale = True
mpr.use_draw_offset_scale = True

behaves differently than (simply reordering the booleans):

mpr = self.gizmos.new(...)
mpr.use_draw_offset_scale = True
mpr.use_draw_scale = True

The boolean properties seem inter-related but it's not clear if that's by-design as it's not documented anywhere.

Is the following a copy-paste error?

RNA_GIZMO_GENERIC_FLAG_RW_DEF(flag_use_draw_offset_scale, flag, WM_GIZMO_DRAW_OFFSET_SCALE);
RNA_GIZMO_GENERIC_FLAG_NEG_RW_DEF(flag_use_draw_scale, flag, WM_GIZMO_DRAW_OFFSET_SCALE);

Exact steps for others to reproduce the error

gizmo_prop_issue.blend

Load the attached .blend (which should have the python open in the script editor already)
Run the supplied python in the editor
Select the default cube and search for the "Test Mesh Symmetry" operator and invoke it
- Notice that it works correctly -- the gizmo remains properly scaled as you zoom in and out in the 3d view
Change the code that was loaded to switch the order of calls (the # DOES NOT WORK!! part)
Repeat above
- Notice that it does not work correctly now -- the gizmo does not scale correctly. Why did a seemingly unrelated change in boolean order create this situation?

**System Information** Operating system: Windows-10-10.0.18362-SP0 64 Bits Graphics card: GeForce GTX 1070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 441.66 **Blender Version** Broken: version: 2.83 (sub 3), branch: master, commit date: 2020-02-14 23:40, hash: `92a56bbe6a` Worked: (optional) **Short description of error** The gist is that: ``` mpr = self.gizmos.new(...) mpr.use_draw_scale = True mpr.use_draw_offset_scale = True ``` behaves differently than (simply reordering the booleans): ``` mpr = self.gizmos.new(...) mpr.use_draw_offset_scale = True mpr.use_draw_scale = True ``` The boolean properties seem inter-related but it's not clear if that's by-design as it's not documented anywhere. Is the following a copy-paste error? ``` RNA_GIZMO_GENERIC_FLAG_RW_DEF(flag_use_draw_offset_scale, flag, WM_GIZMO_DRAW_OFFSET_SCALE); RNA_GIZMO_GENERIC_FLAG_NEG_RW_DEF(flag_use_draw_scale, flag, WM_GIZMO_DRAW_OFFSET_SCALE); ``` **Exact steps for others to reproduce the error** [gizmo_prop_issue.blend](https://archive.blender.org/developer/F8341902/gizmo_prop_issue.blend) - Load the attached .blend (which should have the python open in the script editor already) - Run the supplied python in the editor - Select the default cube and search for the "Test Mesh Symmetry" operator and invoke it - Notice that it works correctly -- the gizmo remains properly scaled as you zoom in and out in the 3d view - Change the code that was loaded to switch the order of calls (the `# DOES NOT WORK!!` part) - Repeat above - Notice that it does not work correctly now -- the gizmo does not scale correctly. Why did a seemingly unrelated change in boolean order create this situation?

Jesse Yurkovich commented

2020-02-15 23:03:12 +01:00

Added subscriber: @deadpin

Richard Antalik commented

2020-02-17 12:16:48 +01:00

Changed status from 'Needs Triage' to: 'Confirmed'

Jesse Yurkovich changed title from ~~bpy.types.Gizmo's boolean scaling related properties seem incorrectly inter-related~~ to bpy.types.Gizmo's boolean scaling properties seem incorrectly inter-related

2020-03-02 07:42:26 +01:00

Evan Wilson commented

2020-03-02 09:26:32 +01:00

Added subscribers: @mano-wii, @ideasman42, @EAW