Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add alpha channel to sprites #23

Merged
merged 5 commits into from
May 5, 2024

Conversation

seb3s
Copy link
Contributor

@seb3s seb3s commented May 4, 2023

This PR adds an optional alpha channel to sprite commands. There is a corresponding PR in scenic to handle that change.

@crertel
Copy link
Contributor

crertel commented Dec 17, 2023

@seb3s , sorry for the delay--would you mind rebasing this, if you still want to go forward? We moved around a bunch of things in the pursuit of Cairo backend.

@seb3s seb3s force-pushed the add-alpha-channel-to-sprites branch from 1d0fe4b to 0dec0b9 Compare December 19, 2023 12:34
@seb3s seb3s force-pushed the add-alpha-channel-to-sprites branch from 0dec0b9 to 621c4a6 Compare December 19, 2023 12:38
@seb3s
Copy link
Contributor Author

seb3s commented Dec 19, 2023

I did some changes, hope this helps (not sure about the cairo modifications)

@seb3s seb3s force-pushed the add-alpha-channel-to-sprites branch from 2e48d49 to 95ed575 Compare December 19, 2023 13:57
@crertel
Copy link
Contributor

crertel commented Apr 14, 2024

Thank you again @seb3s !

We'd need to land this concurrent with some changes to scenic proper, probably over here and possibly elsewhere, I think. Does that sound about right?

@seb3s
Copy link
Contributor Author

seb3s commented Apr 16, 2024

Hello,
I have opened this PR a long time ago for that part:
https://github.com/ScenicFramework/scenic/pull/315/files
cheers,
Sébastien.

@crertel crertel merged commit c68fc00 into ScenicFramework:main May 5, 2024
1 check 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