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

PieChart with Image Value #2328

Closed
wants to merge 1 commit into from
Closed

Conversation

maryamfekri
Copy link

@maryamfekri maryamfekri commented Apr 5, 2017

PieChartDataEntry can be initiated with an image, then image will show beside the value label. (Fixes #2318 )

Changes have made on PieChartDataEntry init functions , and also draw function in PieChartRenderer class to place the image and label in the right position.

screen shot 2017-04-05 at 3 15 38 pm

@maryamfekri
Copy link
Author

what should I with these failing check ? could you please help me to merge these if possible ?

@liuxuan30
Copy link
Member

Wait a minute, have you noticed #1793? Seems the same feature? What's the difference for this one?

@liuxuan30
Copy link
Member

For the test failures, you have to rebase to master, seems you are still using old travis configuration.

@maryamfekri
Copy link
Author

@liuxuan30 it is not same as #1793 , as you can find in my issue ticket , its just about pie chart.

@maryamfekri
Copy link
Author

could you please help me find my travis configuration to update it ? how can I do that then ?

@liuxuan30
Copy link
Member

You just merge master into your branch should be enough

@maryamfekri
Copy link
Author

I couldn't fix this test fail idont know why , but I update another pull request new pull request
@liuxuan30 thanks , could you please merge my pull request

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