Page MenuHome

UI: Tracking: Remove operator options exposed as props
AbandonedPublic

Authored by Aaron Carlisle (Blendify) on Dec 10 2020, 4:05 AM.

Details

Summary

In the movie clip editor, some options of operators are exposed as properties and are shown in the toolbar.
Other areas of Blender do not do this making the movie clip editor break paradigm.
This change is needed to migrate the current toolbar to a new active tool based toolbar

Diff Detail

Repository
rB Blender
Branch
clip_prop_remove (branched from master)
Build Status
Buildable 11655
Build 11655: arc lint + arc unit

Event Timeline

Aaron Carlisle (Blendify) requested review of this revision.Dec 10 2020, 4:05 AM
Aaron Carlisle (Blendify) created this revision.

Merge branch 'master' into clip_prop_remove

Hans Goudey (HooglyBoogly) added inline comments.
source/blender/makesdna/DNA_tracking_types.h
298

Since the padding is a multiple of 8 you should be able to just remove it.

Sergey Sharybin (sergey) requested changes to this revision.Jan 8 2021, 3:26 PM

Other areas of Blender do not do this making the movie clip editor break paradigm.
This change is needed to migrate the current toolbar to a new active tool based toolbar.

I would like to see a more proper design.
Such change has an usability implication, and there needs to be something strong to support such change than a consistency with paradigms.

This revision now requires changes to proceed.Jan 8 2021, 3:26 PM

From feedback, it seems like people don't like the UX of this change, a different solution is used in D10198