-
Notifications
You must be signed in to change notification settings - Fork 605
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(line): line area support isStack options #1705
Conversation
hustcc
commented
Oct 12, 2020
•
edited
Loading
edited
- 折线图支持 isStack
- 面积图支持 isStack
- 双轴图 line 支持 isStack
- 双轴图 legend: false 出现的 异常
- 修复 lint 的 warning
🎊 PR Preview 7d80cc2 has been successfully built and deployed to https://antvis-G2Plot-preview-pr-1705.surge.sh 🕐 Build time: 546.492s 🤖 By surge-preview |
isStack Line 长什么样? |
This pull request fixes 2 alerts when merging 0e618a9 into 9156861 - view on LGTM.com fixed alerts:
|
@@ -41,6 +41,13 @@ export function drawSingleGeometry<O extends { xField: string; yField: string; g | |||
}, | |||
}) | |||
); | |||
|
|||
// 处理 isStack | |||
if (isStack) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是不是直接调 adjust adaptor就好(突然在想双轴图的柱子有没有分组)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
现在是 adaptor/geometry 中的 interval 有处理 adjust,但是 line 没有处理 adjust。这个点是需要重构掉的。