Skip to content

Conversation

DJMcNab
Copy link
Member

@DJMcNab DJMcNab commented Sep 29, 2025

A simpler version of #124, to get in before v0.5.0 (which is currently blocked on #134 anyway).

@DJMcNab DJMcNab requested a review from nicoburns September 29, 2025 15:39
Comment on lines +93 to +94
/// If you were using this with Vello, you should use the (new) `push_clip_layer` function instead.
#[deprecated(note = "Use `push_clip_layer` instead.", since = "0.5.0")]
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to mention that the other alternative is use Mix::Normal?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm going to skip that because:

  1. If you were using this explicitly, then you definitely do want push_clip_layer.
  2. The docs for push_clip_layer is the right place to explain the "child blend layers aren't properly supported" thing
  3. push_clip_layer is so much more performant, that pointing people towards Mix::Normal when they don't need it is a mistake.

@DJMcNab DJMcNab enabled auto-merge September 30, 2025 08:14
@DJMcNab DJMcNab disabled auto-merge September 30, 2025 08:14
@DJMcNab DJMcNab enabled auto-merge September 30, 2025 08:15
@DJMcNab DJMcNab added this pull request to the merge queue Sep 30, 2025
Merged via the queue into linebender:main with commit 1163846 Sep 30, 2025
15 checks passed
@DJMcNab DJMcNab deleted the mix-clip-deprecated branch September 30, 2025 08:24
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