Skip to content

Comments

Add push_clip_layer#30

Merged
nicoburns merged 2 commits intoDioxusLabs:mainfrom
sagudev:clip_layer
Oct 30, 2025
Merged

Add push_clip_layer#30
nicoburns merged 2 commits intoDioxusLabs:mainfrom
sagudev:clip_layer

Conversation

@sagudev
Copy link
Contributor

@sagudev sagudev commented Oct 30, 2025

Peniko's Mix::Clip is deprecated, the correct way is to use push_clip_layer.

Initially I added function without transform argument, but we need to set transform to identity everywhere anyway (backends will then transform clip anyway), so I added it as an argument in second commit (this way users do not need to apply transform on clip by themself as backend will apply transform too).

Part of #29

Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
@sagudev sagudev mentioned this pull request Oct 30, 2025
Copy link
Contributor

@nicoburns nicoburns left a comment

Choose a reason for hiding this comment

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

Looks good - thanks!

@nicoburns nicoburns merged commit acde7a5 into DioxusLabs:main Oct 30, 2025
8 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