Prevent unbalanced SCISSOR_END commands for culled containers#568
Open
depau wants to merge 1 commit intonicbarker:mainfrom
Open
Prevent unbalanced SCISSOR_END commands for culled containers#568depau wants to merge 1 commit intonicbarker:mainfrom
depau wants to merge 1 commit intonicbarker:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change written by AI. PR comment written by me, more about it later.
I found a small issue with Clay not properly closing scissor commands in my layout. I'm using it to write a UI for an embedded display and I saw this glitch.
Before:
After:
I observed the glitch does not occur if I disable culling.
I tried to track it down by comparing the rendering commands and reading the rendering logic but, struggling quite a bit, I thought I'd throw it at an LLM while I was away to eat. To my surprise, it actually fixed the glitch.
So this is slop. I'm not particularly proud of it, I usually prefer to write my code, but while I do really appreciate your amazing effort at writing this library, I don't find layout logic particularly stimulating for myself :)
I cleaned so it's quick to review if it find it useful, it does fix my issue after all. If you'd rather not have this change, I understand.
Anyway, these are the rendering commands before this change with/without culling enabled, then after this change with culling enabled:
Rendering commands
I hope you do understand why this change fixes because if you don't I'm afraid I won't be feeling so comfortable pasting whatever BS Gemini comes up with to explain the changes.
Have a good day!