Page MenuHome

Fix T87983: Motion tracker button tooltips identical
ClosedPublic

Authored by Falk David (filedescriptor) on May 3 2021, 11:48 AM.

Details

Summary

The tooltips for the tracker buttons were all equal (because they all
use the same operator).

This adds the get_description operator callback to the
CLIP_OT_track_markers operator and returns a more descriptive
tooltip for the different buttons.

Diff Detail

Repository
rB Blender

Event Timeline

Falk David (filedescriptor) requested review of this revision.May 3 2021, 11:48 AM
Falk David (filedescriptor) created this revision.

Think this is great to finally distinguish the tooltips. However, I think the strings need to use TIP_ helper. Although, not really my area of expertise. Bastien will know the best!

  • Add TIP_ for translations.
Bastien Montagne (mont29) requested changes to this revision.May 3 2021, 6:45 PM

Indeed TIP_ is mandatory here. Besides picky note about UI messages style, looks fine.

source/blender/editors/space_clip/tracking_ops_track.c
410

We try to avoid third person in UI message, just stick to the infinitive (without the 'to', so imperative?) form: Track the selected...

See also https://wiki.blender.org/wiki/Style_Guide/C_Cpp#UI_Messages

Same below of course.

This revision now requires changes to proceed.May 3 2021, 6:45 PM
  • Change to imperative form
This revision is now accepted and ready to land.May 4 2021, 12:45 PM