Page MenuHome

UI: Improve curve geometry property tooltips
ClosedPublic

Authored by Hans Goudey (HooglyBoogly) on Sep 23 2020, 6:46 PM.

Details

Summary

These tooltips mostly just repeated the property names, which is not
helpful at all. This leaves some space to be much more specific about
what the properties actually do.

Diff Detail

Repository
rB Blender

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.Sep 23 2020, 6:46 PM
Hans Goudey (HooglyBoogly) created this revision.
source/blender/makesrna/intern/rna_curve.c
1657

This is still vague and the bit about object bevel mode is redundant.

1716

Could be more imperative.

Couple comments otherwise looks good

This revision is now accepted and ready to land.Sep 24 2020, 9:29 PM
Ryan Inch (Imaginer) added inline comments.
source/blender/makesrna/intern/rna_curve.c
1716

IMO, the imperative is much harder to read here. I think @Hans Goudey (HooglyBoogly)'s wording is clearer.

Hans Goudey (HooglyBoogly) marked an inline comment as done.Sep 25 2020, 4:50 PM
Hans Goudey (HooglyBoogly) added inline comments.
source/blender/makesrna/intern/rna_curve.c
1657

Mostly agreed, except that "extremities" sort of implies that it includes the extrusion. I found something better.

1716

I agree, I think "that defines" sounds better here.

This revision was automatically updated to reflect the committed changes.
Hans Goudey (HooglyBoogly) marked an inline comment as done.