Sequencer: use Alpha Over blend mode by default
ClosedPublic
Actions

Authored by Jacques Lucke (JacquesLucke) on Apr 25 2019, 2:02 PM.

Details

Reviewers

Sergey Sharybin (sergey)
Richard Antalik (ISS)
Brecht Van Lommel (brecht)

Commits

rBS7c6c5b40cae4: Sequencer: use Alpha Over blend mode by default
rB7c6c5b40cae4: Sequencer: use Alpha Over blend mode by default

Summary

This is a request by @Hjalti Hjálmarsson (hjalti). It was confusing that it is
not consistent currently. He asked me to make alpha over
the default blend mode.

I'm not very used to the sequencer, so I'm not sure if there
are cases when cross makes more sense. In my simple tests
I could not see a difference between cross and alpha over.

Diff Detail

Repository

rB Blender

Branch

sequencer-alpha-over (branched from master)

Build Status

Buildable 3425
Build 3425: arc lint + arc unit

Event Timeline

Jacques Lucke (JacquesLucke) created this revision.Apr 25 2019, 2:02 PM

Harbormaster completed remote builds in B3425: Diff 14911.Apr 25 2019, 2:02 PM

Jacques Lucke (JacquesLucke) edited the summary of this revision. (Show Details)Apr 25 2019, 2:03 PM

Not exactly sure, but cross blend mode will fade alpha channel with opacity, while alpha over will not.

Alpha over as default is ok I guess.

@Brecht Van Lommel (brecht), any objections?

This revision is now accepted and ready to land.Apr 25 2019, 2:53 PM

Fine with me.

For two opaque images, Alpha Over and Cross do the same thing. If there is alpha involved, I expect it's more often needed to do an Alpha Over than to crossfade from one image to the other.

Closed by commit rB7c6c5b40cae4: Sequencer: use Alpha Over blend mode by default (authored by Jacques Lucke (JacquesLucke)). · Explain WhyApr 25 2019, 3:25 PM

This revision was automatically updated to reflect the committed changes.

Richard Antalik (ISS) mentioned this in T69334: Video Sequencer: Very low framerate.Sep 19 2019, 7:09 PM

Richard Antalik (ISS) added a reverting change: D5943: Revert "Sequencer: use Alpha Over blend mode by default".Sep 30 2019, 12:45 AM

Richard Antalik (ISS) added a reverting change: rB64c222ff5857: Fix T69334: VSE Very low framerate.Sep 30 2019, 1:48 AM