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 the ability to set the background color #34

Closed
wants to merge 1 commit into from
Closed

Added the ability to set the background color #34

wants to merge 1 commit into from

Conversation

PGMacDesign
Copy link
Contributor

Added 2 methods for setting the background color of the Elastic view. It should take in either an int color or a String and parse the color. If the string is null or the parsing fails, nothing will be set and it will retain the original color

I created a pull request into the bugfix branch (not master) so you can update after testing if you would like. I tested it myself before pushing and it seems to have no issues that I can find.

Added 2 methods for setting the background color of the Elastic view. It should take in either an int color or a String and parse the color. If the string is null or the parsing fails, nothing will be set and it will retain the original color
@Tibolte
Copy link
Owner

Tibolte commented Jul 29, 2016

You're doing a PR into the wrong branch, don't target restore_state but master. That reminds me I have to delete this old branch :)

@PGMacDesign
Copy link
Contributor Author

Ha. Apologies, I'll make a pull request to master

@PGMacDesign
Copy link
Contributor Author

Updated with new pull:
#35
Thanks!

@PGMacDesign
Copy link
Contributor Author

I see that the merge went through, but it looks like the gadle / maven builds are still the old version. Any chance you could update the file whenever you have some free time? Thanks!

@Tibolte
Copy link
Owner

Tibolte commented Aug 17, 2016

Yes good point I'll try to do that asap and publish a new version.

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.

None yet

2 participants