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

Call setNeedsDisplay() to trigger render noDataText #3198

Merged
merged 1 commit into from
Jan 21, 2018

Conversation

liuxuan30
Copy link
Member

when chart view already rendered, if updating the chartData with nil, it will not rerender, and display the old content
call setNeedsDisplay() here to trigger render noDataText

when chart view already rendered, if updating the chartData with nil, it will not rerender, and display the old content
@liuxuan30 liuxuan30 requested a review from jjatie January 21, 2018 09:19
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3198      +/-   ##
==========================================
- Coverage   22.98%   22.98%   -0.01%     
==========================================
  Files         116      116              
  Lines       15542    15543       +1     
  Branches      272      272              
==========================================
  Hits         3573     3573              
- Misses      11933    11934       +1     
  Partials       36       36
Impacted Files Coverage Δ
Source/Charts/Charts/ChartViewBase.swift 22.61% <0%> (-0.05%) ⬇️

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 43ccf8e...69fd502. Read the comment docs.

@jjatie jjatie added this to the 3.1 milestone Jan 21, 2018
@jjatie jjatie merged commit 4fbeb9d into ChartsOrg:master Jan 21, 2018
@liuxuan30 liuxuan30 deleted the DataNil branch January 21, 2018 23:53
@liuxuan30 liuxuan30 added this to Done in 3.1 release Jan 23, 2018
FreddyZeng added a commit to FreddyZeng/Charts that referenced this pull request Jan 25, 2018
* 'master' of https://github.com/danielgindi/Charts:
  Fixed a duplicated assignment compared with obj-c code. (ChartsOrg#3179)
  Updated README for 3.0.5 (ChartsOrg#3183)
  Added custom text alignment for noData
  Fixes the distance issue between the legend and the horizontal bar chart (Fixes ChartsOrg#2138) (ChartsOrg#2214)
  call setNeedsDisplay() here to trigger render noDataText (ChartsOrg#3198)
FreddyZeng added a commit to FreddyZeng/Charts that referenced this pull request Mar 14, 2018
* master: (55 commits)
  Refactors -[tableView:cellForRowAtIndexPath:] (ChartsOrg#3326)
  fix ChartsOrg#3311. Need one more key for iOS 11 camera roll saving
  revert a mistake, fix ChartsOrg#3299
  add pie chart unit tests (ChartsOrg#3297)
  ChartsOrg#3287: align Objc and Swift demos balloon marker
  update changelog
  Min and Max reset when clearing ChartDataSet (Fixes ChartsOrg#3260)
  Restored old performance (ChartsOrg#3216)
  Support other bundle than main MarkerView.viewFromXib() (ChartsOrg#3215)
  For ChartsOrg#2840. add dataIndex parameter in `highlightValue()` calls (ChartsOrg#2852)
  Balloon Marker indicates position of data (ChartsOrg#3181)
  bump Info.plist version
  Fixed a duplicated assignment compared with obj-c code. (ChartsOrg#3179)
  Updated README for 3.0.5 (ChartsOrg#3183)
  BubbleChart uses correct colour for index now.
  Added custom text alignment for noData
  Fixes the distance issue between the legend and the horizontal bar chart (Fixes ChartsOrg#2138) (ChartsOrg#2214)
  call setNeedsDisplay() here to trigger render noDataText (ChartsOrg#3198)
  添加定制TY的Mark
  添加修改过的Mark到工程中
  ...

# Conflicts:
#	ICXCharts.podspec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
3.1 release
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants