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

Added draw_raw_atlas_texture #646

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jackolantern
Copy link

I'm not sure what the dev process is for this crate, and whether random PRs are okay. I don't think this there is an issue of feature request for this change, but this is something that I very much needed and so I added it an would like to get it upstream.

The change allows - for example - a button to have a graphical label that uses a part of a Texture instead of an Image.

This allows - for example - a button to have a graphical label that uses a part
of a `Texture` instead of an `Image`.
This commit:
- adds a "serde" feature.
- adds the serde crate as an optional dependency, activated by enabling
  the serde feature.
- enables the serde feature of the `glam` dependency when the new serde
  feature is activated
- annotates the `Color` structure with serde's Serialize and Deserialize traits
@not-fl3
Copy link
Owner

not-fl3 commented Sep 16, 2023

Thanks for PR!
Submitting a PR is always OK, no worries about it!

I did not really looked at it yet, but, maybe, it is possible to add a "source" field to the normal DrawTexture, somewhat similar to the one from draw_texture_ex?
It feels more natural, but I did not dived into it yet and it might not be possible.

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