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

Know when user is done interacting with graph #3599

Conversation

dtroupe18
Copy link

Goals ⚽

Easily be able to determine when a user stops interacting (highlighting) with a chart.

Implementation Details 🚧

I made the delegate method optional because it shouldn't have to be implemented. It's just an additional feature that some developers might find useful.

When a user stops a panGesture on a chart the delegate method will call back. This allows for the developer to know when the user stopped highlighting a graph. They can then update the chart accordingly. I used the feature to transition between a cubic (smooth) and linear graph as the user starts and stops highlighting a graph. Similar to the charts in the RobinHood App.

Testing Details 🔍

It's just an optinal callback so didn't add any tests.

@codecov-io
Copy link

Codecov Report

Merging #3599 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3599      +/-   ##
==========================================
- Coverage   29.94%   29.94%   -0.01%     
==========================================
  Files         118      118              
  Lines       13793    13794       +1     
==========================================
  Hits         4130     4130              
- Misses       9663     9664       +1
Impacted Files Coverage Δ
Source/Charts/Charts/ChartViewBase.swift 21.59% <ø> (ø) ⬆️
Source/Charts/Charts/BarLineChartViewBase.swift 26.5% <0%> (-0.03%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4fca1f7...bcab58c. Read the comment docs.

@liuxuan30
Copy link
Member

can you check if latest master fixed yours?

@liuxuan30
Copy link
Member

fixed by #3612

@liuxuan30 liuxuan30 closed this Jan 22, 2019
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.

3 participants