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

Add ability to have the line start or end at any given point #116

Merged
merged 6 commits into from Mar 30, 2015

Conversation

sebastianreloaded
Copy link

This pull requests adds the ability to have the line start or end at any given point.
Now lineChartView: verticalValueForHorizontalIndex: atLineIndex: can return NAN as a value and the line starts at the first non-nan value and ends at the last non-nan value.

i also added an example.

Sebastian Opel added 3 commits October 23, 2014 16:44
JBLineChartView uses numberOfVerticalValuesAtLineIndex to define the number
of points, but subsequently requires that all chartPoints have a value anyway.
This commit allows you to use NAN within the charts data.
@jedborovik
Copy link

Any update on this? This would be awesome to have

@sebastianreloaded
Copy link
Author

@terryworona
hi, do you plan on merging this anytime soon or can i do anything to help?

@terryworona
Copy link
Collaborator

Sorry for not getting to this until now. It's a very good addition. I've made some comments and we need to address the crashes.

As well, can you update the README with some information on NaN support?

@sebastianreloaded
Copy link
Author

hi, thanks for the reply. i'll look into it later this week and fix your points, as well as update the README.

@vocaro
Copy link
Contributor

vocaro commented Mar 7, 2015

Just wanted to say I've been testing this PR for a couple weeks and it's been working great for me. +1 on merging.

@terryworona
Copy link
Collaborator

Cool, just waiting on those last changes then it can be merged. I'll be away from computer for a few weeks, so take your time!

@mingfu
Copy link

mingfu commented Mar 25, 2015

Could really use this functionality as I'm looking for ways to hack around this right now while still using JBChartView

@terryworona
Copy link
Collaborator

Thanks for these fixes. Pulling it in!

terryworona added a commit that referenced this pull request Mar 30, 2015
Add ability to have the line start or end at any given point
@terryworona terryworona merged commit 7046cf7 into Jawbone:master Mar 30, 2015
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

5 participants