vello_hybrid: Incorrect early on on `!self.paint_visible`
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 4.3k
- Forks
- 300
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 61
Description
In Vello Hybrid, in Scene::set_paint, it checks to see if the paint has an alpha of 0.0 and sets paint_visible to false.
Then, in Scene::fill_path and Scene::stroke_path, it has an early out based on that.
This is incorrect though as it depends on the blend mode.
If you have a blend mode that is COPY (or other things), then the operation should still happen.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading Scene::set_paint, Scene::fill_path, and Scene::stroke_path to trace how paint visibility and blend modes interact. Check the early-out behavior for transparent paint with COPY and other blend modes, then add or update coverage so operations that must still apply are not skipped.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- computer-graphics
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100