Skip to content

Conversation

tiaanl
Copy link
Contributor

@tiaanl tiaanl commented Aug 27, 2024

As most of the code refers to types in epaint, it makes sense to use the Color32 alias from epaint as well.

  • I have followed the instructions in the PR template

As most of the code refers to types in epaint, it makes sense to use the
Color32 alias from epaint as well.
@tiaanl tiaanl requested a review from Wumpf as a code owner August 27, 2024 08:23
Copy link
Collaborator

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

sure, same to me either way 🤷

@Wumpf Wumpf merged commit 1ccd056 into emilk:master Sep 9, 2024
18 of 20 checks passed
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
As most of the code refers to types in epaint, it makes sense to use the
Color32 alias from epaint as well.

<!--
Please read the "Making a PR" section of
[`CONTRIBUTING.md`](https://github.com/emilk/egui/blob/master/CONTRIBUTING.md)
before opening a Pull Request!

* Keep your PR:s small and focused.
* The PR title is what ends up in the changelog, so make it descriptive!
* If applicable, add a screenshot or gif.
* If it is a non-trivial addition, consider adding a demo for it to
`egui_demo_lib`, or a new example.
* Do NOT open PR:s from your `master` branch, as that makes it hard for
maintainers to test and add commits to your PR.
* Remember to run `cargo fmt` and `cargo clippy`.
* Open the PR as a draft until you have self-reviewed it and run
`./scripts/check.sh`.
* When you have addressed a PR comment, mark it as resolved.

Please be patient! I will review your PR, but my time is limited!
-->

- [x] I have followed the instructions in the PR template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants