Skip to content

Revert "fix: Use premultiplied WG color space in examples"#43

Closed
hermet wants to merge 1 commit into
mainfrom
revert-42-jay/webgpu-premultiplied-example
Closed

Revert "fix: Use premultiplied WG color space in examples"#43
hermet wants to merge 1 commit into
mainfrom
revert-42-jay/webgpu-premultiplied-example

Conversation

@hermet

@hermet hermet commented Jun 24, 2026

Copy link
Copy Markdown
Member

Reverts #42

Copilot AI review requested due to automatic review settings June 24, 2026 14:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the previous change that switched WebGPU (WgCanvas) example render targets to a premultiplied color space, restoring the prior color space configuration used by the examples.

Changes:

  • Update WgCanvas::target() in the WebGPU multi-canvas example to use tvg::ColorSpace::ABGR8888S.
  • Update WgCanvas::target() in the WebGPU window resize path to use tvg::ColorSpace::ABGR8888S.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/MultiCanvas.cpp Reverts the WebGPU offscreen target color space to ABGR8888S in the multi-canvas example loop.
src/Example.h Reverts the WebGPU window canvas target color space to ABGR8888S on resize.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hermet hermet closed this Jun 24, 2026
@hermet hermet added the invalid This doesn't seem right label Jun 24, 2026
@hermet hermet deleted the revert-42-jay/webgpu-premultiplied-example branch July 9, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants