Skip to content

anyrender_skia: respect CTM when blurring box shadows - #75

Merged
nicoburns merged 1 commit into
mainfrom
devin/1785852683-blur-respect-ctm
Aug 4, 2026
Merged

anyrender_skia: respect CTM when blurring box shadows#75
nicoburns merged 1 commit into
mainfrom
devin/1785852683-blur-respect-ctm

Conversation

@nicoburns

Copy link
Copy Markdown
Member

Summary

draw_box_shadow created its blur MaskFilter with respect_ctm = false, so the blur sigma was interpreted in device pixels regardless of the canvas transform — on a HiDPI-scaled or CSS-transformed canvas, shadows blurred too little/too much. Flip it to true so the sigma is transformed by the CTM (matching Chrome's box-shadow behavior). This doesn't affect Skia's analytic blurred-rrect fast path.

Link to Devin session: https://dioxus.staging.devinenterprise.com/sessions/f10beda458004678a1c07c8b30d351ed
Requested by: @nicoburns

@nicoburns nicoburns self-assigned this Aug 4, 2026
@staging-devin-ai-integration

Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@nicoburns
nicoburns merged commit a9df194 into main Aug 4, 2026
9 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.

1 participant