Page MenuHome
Differential D4506

Fix strip text hiding behind scrollers
ClosedPublic
Actions

Authored by Richard Antalik (ISS) on Mar 12 2019, 11:41 PM.

Details

Reviewers
Brecht Van Lommel (brecht)
Commits
rBS3d1138840c9e: Fix strip text hiding behind scrollers
rB3d1138840c9e: Fix strip text hiding behind scrollers
Summary

Fix strip text hiding behind scrollers

Diff Detail

Repository
rB Blender

Event Timeline

Richard Antalik (ISS) created this revision.Mar 12 2019, 11:41 PM
Brecht Van Lommel (brecht) added a comment.Mar 13 2019, 2:13 PM

Can this just use V2D_SCROLL_WIDTH_TEXT? Computing scrollers for every strip drawn seems a little expensive and complicated.

Adding a little extra padding seems fine, it's not ideal the way it's right at the edge now.

Brecht Van Lommel (brecht) requested changes to this revision.Mar 13 2019, 2:13 PM
This revision now requires changes to proceed.Mar 13 2019, 2:13 PM
Richard Antalik (ISS) updated this revision to Diff 14164.Mar 13 2019, 9:45 PM
Richard Antalik (ISS) edited the summary of this revision. (Show Details)

Thanks, I wasn't looking hard enough :)

Richard Antalik (ISS) edited the summary of this revision. (Show Details)Mar 13 2019, 9:45 PM
Brecht Van Lommel (brecht) accepted this revision.Mar 14 2019, 3:41 PM
This revision is now accepted and ready to land.Mar 14 2019, 3:41 PM
Closed by commit rB3d1138840c9e: Fix strip text hiding behind scrollers (authored by Richard Antalik (ISS)). · Explain WhyMar 18 2019, 9:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
space_sequencer/
6 lines

Diff 14265

View Options

source/blender/editors/space_sequencer/sequencer_draw.c

Loading...