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

feat: multi layer 支持配置 annotations & label #1913

Merged
merged 9 commits into from Nov 14, 2020
Merged

Conversation

visiky
Copy link
Member

@visiky visiky commented Nov 14, 2020

feat

  • 🧪 实验室 multi-views,每个 view 都允许配置 annotations(后续继续增强 interactions 等 API)
  • geometry-base 增加 label API,label 通道作用于几何对象上;
    ① multi-views 中每个图层的每个geometry需要支持独立配置label;
    ② 另外plots中的使用 geometry 通道的之后,会再次使用 label 通道,故会直接覆盖,不产生影响;
    ③ 但后续可以考虑直接使用 geometry 通道来完成 label 通道,以及可以满足 @zqlu 提到的 line plots 的 point 是否支持配置 label

背景
🧪 实验室实践中发现的问题
image

checklist

  • testcases

@github-actions
Copy link
Contributor

github-actions bot commented Nov 14, 2020

🎊 PR Preview e21a618 has been successfully built and deployed to https://antvis-G2Plot-preview-pr-1913.surge.sh

🕐 Build time: 580.704s

🤖 By surge-preview

@visiky visiky requested a review from lessmost November 14, 2020 00:10
@coveralls
Copy link

coveralls commented Nov 14, 2020

Pull Request Test Coverage Report for Build 362696299

  • 11 of 11 (100.0%) changed or added relevant lines in 3 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.01%) to 96.057%

Files with Coverage Reduction New Missed Lines %
src/adaptor/geometries/base.ts 1 98.47%
Totals Coverage Status
Change from base Build 361368104: 0.01%
Covered Lines: 3817
Relevant Lines: 3870

💛 - Coveralls

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are accessibility issues in these changes.

__tests__/unit/plots/multi-view/multi-donut-spec.ts Outdated Show resolved Hide resolved
examples/case/multi-views/demo/donut.ts Outdated Show resolved Hide resolved
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@hustcc hustcc merged commit 41b7cff into master Nov 14, 2020
@hustcc hustcc deleted the multi-layer/geometry branch November 14, 2020 04:06
jinhuiWong pushed a commit to jinhuiWong/G2Plot that referenced this pull request Nov 15, 2020
* refactor(docs): 饼图 & 雷达图 & 瀑布图走查

* feat(multi-layer): 实验室多图层,允许每个图层配置 annotations & 每个图层的每个geometry配置label

* fix(geometry-adaptor/label): geometry label 通道与子弹图label通道的适配

* feat(geometry-adaptor/label): label 通道支持透传 fields 映射

* fix(geometry-adaptor): geometry adaptor 的 size 通道怪怪的

* feat(demo): 添加一个动态的多view组合 demo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants