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.

Sign up for GitHub

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

Jump to bottom

feat(tooltip): add tooltip.showNil #3216

Merged
merged 2 commits into from
Jan 19, 2021
Merged

feat(tooltip): add tooltip.showNil #3216

merged 2 commits into from
Jan 19, 2021

Conversation

hustcc
Copy link
Member

@hustcc hustcc commented Jan 19, 2021 •
edited
Loading

@hustcc hustcc requested a review from visiky January 19, 2021 08:37
@github-actions
Copy link

github-actions bot commented Jan 19, 2021 •
edited
Loading

😭 Deploy PR Preview eaaa11b failed. Build logs

🤖 By surge-preview

@coveralls
Copy link

Pull Request Test Coverage Report for Build 495738594

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • 7 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.04%) to 93.514%
Files with Coverage Reduction New Missed Lines %
src/chart/controller/gesture.ts 7 82.14%
Totals Coverage Status
Change from base Build 486979465: -0.04%
Covered Lines: 10822
Relevant Lines: 11228
💛 - Coveralls

@visiky visiky merged commit edc1cee into master Jan 19, 2021
@visiky visiky deleted the feat-tooltip-showNil branch January 19, 2021 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@visiky visiky visiky approved these changes

Assignees
No one assigned
Labels
PR: merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tooltip默认过滤掉value为null 的内容，是否能够增加配置
3 participants
@hustcc @coveralls @visiky