Page MenuHome

Remove workaround for drawing the rotation gizmo ('ED_gizmo_types_dial_3d_draw_util')
ClosedPublic

Authored by Germano Cavalcante (mano-wii) on Nov 12 2020, 6:43 PM.

Details

Summary

Since {648350e456490f8d6258e7de9bf94d3a6a34dbb2}, transform gizmos are no longer hidden during transform.

The same can be observed for rotation gizmos.

However, as a workaround for these rotation gizmos, there was already a drawing utility running.

With the gizmo and the utility this drawing is now being done twice.

So remove the utility/workaround and update the gizmo accordingly.

Diff Detail

Repository
rB Blender
Branch
master
Build Status
Buildable 11244
Build 11244: arc lint + arc unit

Event Timeline

Germano Cavalcante (mano-wii) requested review of this revision.Nov 12 2020, 6:43 PM
Germano Cavalcante (mano-wii) created this revision.
Germano Cavalcante (mano-wii) retitled this revision from Transform: Remove the "wm.h" header to Remove workaround for drawing the rotation gizmo (`ED gizmo types dial_3d_draw_util`).Jun 7 2022, 5:12 AM
Germano Cavalcante (mano-wii) edited the summary of this revision. (Show Details)
Germano Cavalcante (mano-wii) retitled this revision from Remove workaround for drawing the rotation gizmo (`ED gizmo types dial_3d_draw_util`) to Remove workaround for drawing the rotation gizmo ('ED_gizmo_types_dial_3d_draw_util').Jun 7 2022, 5:20 AM

Accepting with some minor requests.

source/blender/editors/transform/transform_gizmo_3d.c
1274–1281

No need for a struct qualifier, all args can be const besides r_mat_basis

1337

UNUSED(gz)

1343

UNUSED(gz)

This revision is now accepted and ready to land.Jun 7 2022, 5:25 AM