Skip to content

Fix Normal() not factoring in opacity#41

Merged
AristurtleDev merged 1 commit intoAristurtleDev:developfrom
guptohkl:fix/normal-blend-alpha
Apr 3, 2026
Merged

Fix Normal() not factoring in opacity#41
AristurtleDev merged 1 commit intoAristurtleDev:developfrom
guptohkl:fix/normal-blend-alpha

Conversation

@guptohkl
Copy link
Copy Markdown
Contributor

@guptohkl guptohkl commented Apr 3, 2026

Prerequisites

  • I have verified that there are no existing pull requests that would overlap with this pull request.
  • I have verified that I am following the guidelines as outlined in this project's contribution policy
  • I Have verified that this pull request adheres to this project's code of conduct.
  • I have written a descriptive title for this pull request.
  • I have provided appropriate test coverage were applicable.

Description

The alpha calculation in Normal() did not factor in opacity, causing layer and cel opacity to render incorrectly against transparent layers.

Related Issue Ticket Numbers

N/A

The alpha calculation in Normal() did not factor in opacity, causing
layer and cel opacity to render incorrectly against transparent layers.
Copy link
Copy Markdown
Owner

@AristurtleDev AristurtleDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @guptohkl

@AristurtleDev
Copy link
Copy Markdown
Owner

This issue was originally pointed out in discord conversation https://discord.com/channels/786590682566754334/786591601237557258/1489468260951785493

@AristurtleDev
Copy link
Copy Markdown
Owner

CI failure is due to artifact upload issue, the build is good

@AristurtleDev AristurtleDev merged commit 399577c into AristurtleDev:develop Apr 3, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants