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

I changed the included color functions to consts and added a "TRANSPARENT" color #657

Merged
merged 4 commits into from
Dec 23, 2015

Conversation

BenMcH
Copy link
Contributor

@BenMcH BenMcH commented Dec 23, 2015

I also updated the examples to work with the new changes made.
I made these changes in accordance with issue #638. Everything is working properly.

use conrod::{Button, Colorable, Labelable, Positionable, Sizeable, Tabs, Text, WidgetMatrix};

// Construct our main `Canvas` tree.
Canvas::new().flow_down(&[
(HEADER, Canvas::new().color(blue()).pad_bottom(20.0)),
(HEADER, Canvas::new().color(::conrod::color::BLUE).pad_bottom(20.0)),
Copy link
Contributor

Choose a reason for hiding this comment

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

If possible, could you import the color module and instantiate these colors like color::BLUE instead of ::conrod::color::BLUE?

@mitchmindtree
Copy link
Contributor

Nice one, thanks a lot for this, really appreciate it! If you could make the change I suggested in the other comment, I'll merge this 😸

@BenMcH
Copy link
Contributor Author

BenMcH commented Dec 23, 2015

I made that change! Feel free to merge if you see fit. Else, tell me what to change! 😃

@mitchmindtree
Copy link
Contributor

LGTM! 👍

Travis is failing due to a regression in the latest nightly, but it's building fine on stable so will merge :)

I'll increment the version and publish this to crates.io a little later on.

mitchmindtree added a commit that referenced this pull request Dec 23, 2015
I changed the included color functions to consts and added a "TRANSPARENT" color
@mitchmindtree mitchmindtree merged commit 935ee46 into PistonDevelopers:master Dec 23, 2015
@mitchmindtree
Copy link
Contributor

These breaking changes have been published to crates.io under version 0.29.0, along with the breaking changes in #662.

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