Fix T77596: UI: Move use_even_offset to main solidify modifier panel
ClosedPublic
Actions

Authored by Hans Goudey (HooglyBoogly) on Jun 9 2020, 3:43 PM.

Details

Reviewers

Aaron Carlisle (Blendify)
William Reynish (billreynish)

Maniphest Tasks

T77596: UI: Solidify Modifier: Even Thickness is not related to normals

Commits

rB73f8dd0f4cb5: Fix T77596: UI: Correct "offset even" location in solidify modifier

Summary

This only applies in the "simple" solidify mode, but it definitely shouldn't be in
the normals panel. Putting it where "use_flat_faces" is for the "complex" mode seems
to fit well.

Diff Detail

Repository

rB Blender

Branch

use-even-offset-move (branched from master)

Build Status

Buildable 8460
Build 8460: arc lint + arc unit

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.Jun 9 2020, 3:43 PM

Hans Goudey (HooglyBoogly) created this revision.

Harbormaster completed remote builds in B8460: Diff 25673.Jun 9 2020, 3:43 PM

Hans Goudey (HooglyBoogly) added a task: T77596: UI: Solidify Modifier: Even Thickness is not related to normals.Jun 9 2020, 3:44 PM

William Reynish (billreynish) accepted this revision.Jun 9 2020, 3:45 PM

This revision is now accepted and ready to land.Jun 9 2020, 3:45 PM

Closed by commit rB73f8dd0f4cb5: Fix T77596: UI: Correct "offset even" location in solidify modifier (authored by Hans Goudey (HooglyBoogly)). · Explain WhyJun 15 2020, 2:23 PM

This revision was automatically updated to reflect the committed changes.

Hans Goudey (HooglyBoogly) added a commit: rB73f8dd0f4cb5: Fix T77596: UI: Correct "offset even" location in solidify modifier.

Revision Contents
Changeset List

Path

Size

source/

blender/

modifiers/

intern/

MOD_solidify.c

4 lines

Diff 25673

View Options

source/blender/modifiers/intern/MOD_solidify.c

Fix T77596: UI: Move use_even_offset to main solidify modifier panelClosedPublicActions