We no longer need the Empty workaround since rB2e70d5cb980e
Also fix UI error when Track To constraint was renamed
Differential D15471
Add camera rigs: use bone subtarget for DOF Authored by Damien Picard (pioverfour) on Jul 15 2022, 10:51 PM.
Details We no longer need the Empty workaround since rB2e70d5cb980e Also fix UI error when Track To constraint was renamed
Diff Detail Event TimelineComment Actions Hi Damien, That very change was on my never ending to do list. Glad someone got to it! The biggest one I can remember though was to actually adjust the naming convention to match with rigify. I've had a change of heart and I do think that it is much better to be consistent with the rest of Blender. I don't know how the reviewing process works but I will download this patch and test everything - but that will most likely be on the weekend. Thanks for your work! Comment Actions Take your time! For review, downloading and testing the patch is indeed the most important part! That, and checking that the new behaviour, code and UI fit in with your vision for the add-on. Comment Actions Hi Damien, I seem to be having trouble applying the patch. "patch -p1 > D15471.diff" Then the terminal just hangs (no output) I don't know what I've done wrong. Comment Actions It looks like: patch -p1 < D15471.diff works as well (the file is passed to the input of patch with a <). Comment Actions Hi Damien, It looks like it is all working as expected although I realise that there probably needs to be a way to disable the DoF on the bone once it has been enabled. I don't think this existed in the old version either, but the workaround was to delete the dof_empty (something you could do in the viewport) Maybe the UI panel should expose the camera focus object (if it exists) so the user can remove it by clicking the X Comment Actions You’re right, that makes a lot of sense. So now you can always either call the operator to setup the bone, or set the object and bone yourself, and you can also remove either. The UI was tweaked to reflect that. I also noticed there was still the Empty icon for the operator so I removed it. Comment Actions Looking good Damien. Let me ping @Brendon Murphy (meta-androcto) to ask him what the proper way is to get this update committed. Comment Actions I’m not sure Brendon is active in Blender development anymore. Besides, if you agree with it I can push this myself since I’ve got commit rights for the add-ons repo. |