Blender Git Commit Log
Git Commits -> Revision 6154aa1
Revision 6154aa1 by Philipp Oeser (master) January 9, 2021, 14:02 (GMT) |
Cleanup: remove redundant decorator for eevee 'overscan_size' All eevee scene properties are not animatable due to `RNA_define_animate_sdna(false)` so there is no need for an animation decorator here (would not show anyways). Not sure if setting use_property_decorate to False and then manually adding one again was a hack in rB7fc60bff14a6 to get the alignment right, but seems to work good now without it. Note: there also seems to be an inconsistency here generally: - Eevee reserves room for an animation decorator in almost every panel (except for "Sampling" and "Indirect Lighting") even though almost none are animatable (except for stuff in "Hair" and "Film"). This looks nice if multiple panels are expanded (except for mentioned "Sampling" and "Indirect Lighting" -- maybe these should also reserve the room?) - Cycles does not use animation decorators at all (even though pretty much everything is animatable here -- maybe these should also use animation decorators?) - Then there is also the 'shared' "Grease Pencil", "Freestyle" and "Colormanagement" -- these dont use animation decorators, but some stuff is animatable... Regarding the note: not sure what the guidelines here are, if this is isolated to each panel then there is lots of stuff for eevee that could set use_property_decorate to False, if multiple open panels are considered (for nice visual consistency between them), then there is romm for improvement elsewhere, too. Maniphest Tasks: T81411 Differential Revision: https://developer.blender.org/D9164 |
Commit Details:
Full Hash: 6154aa1529b77a9ede5afdac148048e6eb02dbf3
Parent Commit: fa82a15
Lines Changed: +0, -2
1 Modified Path:
/release/scripts/startup/bl_ui/properties_render.py (+0, -2) (Diff)