Page MenuHome

UI: Use property split layout for add torus operator
ClosedPublic

Authored by Hans Goudey (HooglyBoogly) on Aug 29 2020, 12:09 AM.

Details

Summary

This is just an operator that I missed earlier in D8326.

I also moved the Generate UVs, Align, Location, and Rotation properties to the bottom to
be consistent with all other add mesh operators.

BeforeAfter

Diff Detail

Repository
rB Blender

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.Aug 29 2020, 12:09 AM

Nice one!

Perhaps outside the scope of this patch, but when Align is set to View or 3D Cursor, the Rotation values are ignored, maybe we shouldn't even show the sliders then?

This revision is now accepted and ready to land.Sep 2 2020, 5:00 PM

Perhaps outside the scope of this patch, but when Align is set to View or 3D Cursor, the Rotation values are ignored, maybe we shouldn't even show the sliders then?

The situation there is definitely a bit odd. With the two latter options you can still change the rotation actually, then if you set align to view or cursor again it will reset it. It's probably not ideal, but keeping the rotation buttons visible does make it faster to change it if your default is view or cursor.