Merge lp://staging/~mbranton/libopenshot/alpha-channel-fix into lp://staging/libopenshot/0.0
Proposed by
matt
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://staging/~mbranton/libopenshot/alpha-channel-fix |
| Merge into: | lp://staging/libopenshot/0.0 |
| Diff against target: |
12 lines (+1/-1) 1 file modified
src/Color.cpp (+1/-1) |
| To merge this branch: | bzr merge lp://staging/~mbranton/libopenshot/alpha-channel-fix |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| OpenShot Code | Pending | ||
|
Review via email:
|
|||
Description of the change
Color::GetColorHex was returning a value that dropped the alpha channel, resulting in incorrect colors being passed to flood fill. Since it explicitly gets the alpha channel only to immediately discard it I think this is a bug. It also has the side effect of fixing the timeline so that if you specify an alpha channel for the background color it will be preserved if the color is non black.
To post a comment you must log in.
Unmerged revisions
- 561. By Matt Branton <email address hidden>
-
Set color format so that alpha can be preserved on flood fills
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
