Skip to content

Conversation

arcsin1
Copy link
Contributor

@arcsin1 arcsin1 commented Sep 8, 2020

基于上个版本:

  • 新改动子弹图 (用新的api替代)
  • 新增docs
  • 完善测试用例

@auto-add-label auto-add-label bot added the enhancement New feature or request label Sep 8, 2020

默认配置: 无

### bulletLabel
Copy link
Member

Choose a reason for hiding this comment

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

一定要这么取名吗?不能直接 label ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😳 是为了统一的那个bulletStyle类似 尴尬了 觉得不合适的我等等可以修复掉

Copy link
Member

Choose a reason for hiding this comment

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

不合适,其他 plot 都是 label~

Copy link
Member

Choose a reason for hiding this comment

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

看看好不好改,如果不好改先合并了,然后下一个分支写 demo 的时候改掉吧。 @arcsin1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好改 我马上准备提交了

@@ -1,41 +1,41 @@
import { BulletOptions, BulletData } from './types';
import { BulletOptions } from './types';

type TransFormData = {
Copy link
Member

Choose a reason for hiding this comment

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

nit: TransformData。Transform 是一个单词

@hustcc hustcc merged commit cbfe496 into antvis:master Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request PR: merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants