-
Notifications
You must be signed in to change notification settings - Fork 602
feat: 新改动子弹图和新增docs #1545
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: 新改动子弹图和新增docs #1545
Conversation
增加histogram的文档
examples/bullet/basic/API.en.md
Outdated
|
||
默认配置: 无 | ||
|
||
### bulletLabel |
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.
一定要这么取名吗?不能直接 label ?
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.
😳 是为了统一的那个bulletStyle类似 尴尬了 觉得不合适的我等等可以修复掉
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.
不合适,其他 plot 都是 label~
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.
看看好不好改,如果不好改先合并了,然后下一个分支写 demo 的时候改掉吧。 @arcsin1
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.
好改 我马上准备提交了
src/plots/bullet/utils.ts
Outdated
@@ -1,41 +1,41 @@ | |||
import { BulletOptions, BulletData } from './types'; | |||
import { BulletOptions } from './types'; | |||
|
|||
type TransFormData = { |
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.
nit: TransformData。Transform 是一个单词
基于上个版本: