mirror/aseprite
1
0
Fork 0
mirror of https://github.com/aseprite/aseprite.git synced 2025-03-30 04:20:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix edge case w/alpha compositing on indexed images

Browse Source 
On indexed images, if we paint with alpha compositing and opacity=255,
we can act just like the simple ink, copying the color (the original
index). If we go through the process of color blending, we might loss
the index in sprites that contain palettes with the same color two or
more times.
This commit is contained in:
David Capello 2020-05-21 20:36:04 -03:00
parent c6720b543a
commit 60c54a3242
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/app/tools/inks.h
View File

@ -114,9 +114,11 @@ public:
}
else {
switch (m_type) {
case Simple: {
case Simple:
case AlphaCompositing: {
bool opaque = false;
// Opacity is set to 255 when InkType=Simple in ToolLoopBase()
if (loop->getOpacity() == 255 &&
// The trace policy is "overlap" when the dynamics has
// a gradient between FG <-> BG
@ -134,8 +136,15 @@ public:
opaque = (graya_geta(color) == 255);
break;
case IMAGE_INDEXED:
color = get_current_palette()->getEntry(color);
opaque = (rgba_geta(color) == 255);
// When we paint over a transparent layer with the
// mask color, we have to use the
// TransparentInkProcessing anyway (cannot use the
// fast path with CopyInkProcessing).
if (loop->getLayer()->isBackground() ||
color != loop->sprite()->transparentColor()) {
color = get_current_palette()->getEntry(color);
opaque = (rgba_geta(color) == 255);
}
break;
}
}
@ -147,9 +156,6 @@ public:
setProc(get_ink_proc<TransparentInkProcessing>(loop));
break;
}
case AlphaCompositing:
setProc(get_ink_proc<TransparentInkProcessing>(loop));
break;
case Copy:
setProc(get_ink_proc<CopyInkProcessing>(loop));
break;