UI: Clarify the property name of "F-Curve Visibility"
ClosedPublic
Actions

Authored by Hans Goudey (HooglyBoogly) on Jan 7 2021, 6:52 AM.

Details

Reviewers

Luciano Muñoz Sessarego (looch)
Sybren A. Stüvel (sybren)

Group Reviewers

Animation & Rigging

Maniphest Tasks

T82587: Bad naming on F-curve visibility option in User Preferences > Animation

Commits

rBb9e02eace69e: UI: Clarify the property name of "F-Curve Visibility"

Summary

"F-Curve" visibility is a bad UI label for a property that only affects
the display of unselected F-Curves. This commit clarifies the property
name by making it more specific with the word "Unselected", and by using
the word "Opacity".

"F-Curve" is redundant in the UI label anyway because it is included
in the panel title.

Resolves T82587

Diff Detail

Repository

rB Blender

Branch

unselected-opacity-bad-name (branched from master)

Build Status

Buildable 12028
Build 12028: arc lint + arc unit

Event Timeline

Hans Goudey (HooglyBoogly) created this revision.Jan 7 2021, 6:52 AM

Harbormaster completed remote builds in B12028: Diff 32503.Jan 7 2021, 6:52 AM

Hans Goudey (HooglyBoogly) requested review of this revision.Jan 7 2021, 6:52 AM

Hans Goudey (HooglyBoogly) added a reviewer: Animation & Rigging.Jan 11 2021, 10:39 PM

Personally I wouldn't mind when the label would include the word "FCurve". To me "Opacity Of Unselected FCurves" would be best, but given how the panel unnecessarily cuts off an even shorter label already, I think the patch is fine as it is: