Page MenuHome
Differential D7788

Clarify autokeyframe tootip
ClosedPublic
Actions

Authored by Philipp Oeser (lichtwerk) on May 19 2020, 1:50 PM.

Details

Reviewers
Sergey Sharybin (sergey)
William Reynish (billreynish)
Commits
rB059f79bdd745: Clarify autokeyframe tootip
Summary

This applies to masks as well...

Spotted while looking into T76872.

Diff Detail

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

Event Timeline

Philipp Oeser (lichtwerk) requested review of this revision.May 19 2020, 1:50 PM
Philipp Oeser (lichtwerk) created this revision.
Harbormaster completed remote builds in B8118: Diff 24920.May 19 2020, 1:50 PM
Sergey Sharybin (sergey) accepted this revision.May 19 2020, 3:27 PM

This applies to masks as well...

What is it you are not telling me? From the meaning and intention use of ellipsis:

Ellipsis indicates an intentional omission of a word, sentence, or whole section from a text without altering its original meaning.

Don't omit anything. And here using fullstop will be enough and even more clear.

Other than that LGTM.

This revision is now accepted and ready to land.May 19 2020, 3:27 PM
Closed by commit rB059f79bdd745: Clarify autokeyframe tootip (authored by Philipp Oeser (lichtwerk)). · Explain WhyMay 19 2020, 6:54 PM
This revision was automatically updated to reflect the committed changes.
Philipp Oeser (lichtwerk) added a commit: rB059f79bdd745: Clarify autokeyframe tootip.
Philipp Oeser (lichtwerk) added a comment.May 19 2020, 6:57 PM
In D7788#187846, @Sergey Sharybin (sergey) wrote:

This applies to masks as well...

What is it you are not telling me? From the meaning and intention use of ellipsis:

Ellipsis indicates an intentional omission of a word, sentence, or whole section from a text without altering its original meaning.

Don't omit anything. And here using fullstop will be enough and even more clear.

Other than that LGTM.

:)

Thx checking! (almost did the ellipsis again -- tempted another time actually. stop!)

Revision Contents
Changeset List

PathSize
source/
blender/
makesrna/
intern/
4 lines

Diff 24920

View Options

source/blender/makesrna/intern/rna_scene.c

Loading...