Skip to content

Conversation

@chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 7, 2024

The specification currently says that the globalAlpha should be
applied on the result of the context's filter. This however leads to
an unexpected behavior because if the filter is a drop-shadow, we
can't see the shadow through the foreground even though the foreground
is made transparent by the globalAlpha.

This is not what Chrome and Firefox do. These browsers instead apply
the globalAlpha on the source of the filter, therefore making the
foreground transparent.

The specification issue is addressed in:
whatwg/html#10674

Change-Id: I3832c3cf6390933770a08669f62e9b1b29caccf8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5904448
Reviewed-by: Aaron Krajeski <[email protected]>
Commit-Queue: Jean-Philippe Gravel <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1365599}

The specification currently says that the globalAlpha should be
applied on the result of the context's filter. This however leads to
an unexpected behavior because if the filter is a drop-shadow, we
can't see the shadow through the foreground even though the foreground
is made transparent by the globalAlpha.

This is not what Chrome and Firefox do. These browsers instead apply
the globalAlpha on the source of the filter, therefore making the
foreground transparent.

The specification issue is addressed in:
whatwg/html#10674

Change-Id: I3832c3cf6390933770a08669f62e9b1b29caccf8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5904448
Reviewed-by: Aaron Krajeski <[email protected]>
Commit-Queue: Jean-Philippe Gravel <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1365599}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit ac35e3f into master Oct 8, 2024
22 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5904448 branch October 8, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants