Page MenuHome

Curves: Disable Curve Normals by default
ClosedPublic

Authored by Antonio Vazquez (antoniov) on May 16 2020, 7:02 PM.

Details

Summary

This patch just disable the curve Normals by default.

This is a change related to D7754, but I decided to put in different patch to keeep reference of the changes.

Diff Detail

Repository
rB Blender

Event Timeline

Antonio Vazquez (antoniov) requested review of this revision.May 16 2020, 7:02 PM
Antonio Vazquez (antoniov) created this revision.

Not sure if I must patch old files or only templates.

Agree with this change.

I don’t think you need to patch old files.

This revision is now accepted and ready to land.May 16 2020, 8:01 PM

With the current versioning, the option will be disabled for all existing files, even if users actually wanted to have it enabled. I'd just change it in the bundled startup.blend's, https://developer.blender.org/diffusion/B/browse/master/source/blender/blenloader/intern/versioning_defaults.c$185.

@Julian Eisel (Severin) Yes, I'm going to change the patch..I don't like do versioning for all files.. I'm waiting for an asnswer from Campebll before update the patch.

Still missing the change in startup.blend file. I think is better to do the change in a Linux machine than my Windows machine..it's just open it, disable the option and save it.

After talking with @Campbell Barton (campbellbarton), we don't need replace any blend file, because if you load factory settings the Normals are disabled, so I think we can commit this change.

Julian Eisel (Severin) accepted this revision.EditedMay 18 2020, 11:19 AM

The change in versioning_defaults.c does exactly that: It updates values read from the startup.blend's (each template can have its own startup.blend).