Page MenuHome

UI: Graph Editor: Draw Handles Lines With Smoothing
ClosedPublic

Authored by Hans Goudey (HooglyBoogly) on May 15 2020, 4:06 AM.

Details

Summary

Enabling "Use High Quality Display" in the graph editor currently enables AA for the curves, but the handles are still ugly.

With smoothing they look a lot better and fit in better with the rest of the UI.

BeforeAfter

This also makes the handle lines slightly thicker, which I see as a benefit as well.

I may be missing something, but this seems like a simple quick fix for an oversight.

Diff Detail

Repository
rB Blender
Branch
graph-handle-smooth (branched from master)
Build Status
Buildable 8056
Build 8056: arc lint + arc unit

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.May 15 2020, 4:06 AM
Hans Goudey (HooglyBoogly) created this revision.
source/blender/editors/space_graph/graph_draw.c
346

The added lines mirror code elsewhere in the file, although I got the same results without GPU_blend(true); on the default theme.

Brecht Van Lommel (brecht) requested changes to this revision.May 15 2020, 4:35 PM
Brecht Van Lommel (brecht) added inline comments.
source/blender/editors/space_graph/graph_draw.c
346–349

Line smooth and blend should be disabled again at the end of the function, as is done in other places in this file.

This revision now requires changes to proceed.May 15 2020, 4:35 PM
source/blender/editors/space_graph/graph_draw.c
346

Note the code that calls this function is:

GPU_blend(true);
draw_fcurve_handles(sipo, fcu);
GPU_blend(false);

Removing the GPU_blend there and moving it into this function seems more consistent to me.

  • Merge branch 'master' into graph-handle-smooth
  • Remove GPU_blend calls outside function, set false after
Hans Goudey (HooglyBoogly) marked 2 inline comments as done.May 15 2020, 8:00 PM

Right, that was sloppy, sorry. Should be fine now.

This revision is now accepted and ready to land.May 15 2020, 8:55 PM