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

增加饼、环图文档 ,同时补全饼图能力 #1411

Merged
merged 5 commits into from
Aug 10, 2020
Merged

增加饼、环图文档 ,同时补全饼图能力 #1411

merged 5 commits into from
Aug 10, 2020

Conversation

visiky
Copy link
Member

@visiky visiky commented Aug 9, 2020

  • 增加 pie-inner label 类型
    • 视后续情况,落入 G2 层
  • 抽取 legend 通用 adaptor,修复 饼图 legend 关闭失效
    • testcases
  • 增加饼、环 examples & api 文档初稿
  • 环图中心文本指标卡调整

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.

examples/pie/basic/API.zh.md Show resolved Hide resolved
@visiky visiky requested review from hustcc, lxfu1 and lessmost and removed request for hustcc August 9, 2020 14:06
@visiky visiky added this to the v2 milestone Aug 9, 2020
@visiky visiky changed the title 增加饼图文档 ,同时补全饼图能力 增加饼、环图文档 ,同时补全饼图能力 Aug 9, 2020
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.

examples/pie/basic/API.en.md 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.

There are accessibility issues in these changes.

examples/pie/basic/API.en.md Show resolved Hide resolved
examples/pie/basic/API.zh.md Show resolved Hide resolved
}
return isNil(actualOffset) || actualOffset > 0 ? -(radius - innerRadius) * 0.3 : actualOffset;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

直接可以搞到 G2 去啊! @simaQ

Copy link

Choose a reason for hiding this comment

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

对的呢,搞过来呀!

Copy link
Member Author

Choose a reason for hiding this comment

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

确定给 G2 label offset 增加 string 百分比类型是吧?(仅供 pie 使用)

if (isString(actualOffset)) {
actualOffset = (radius - innerRadius) * parsePercentageToNumber(actualOffset);
}
return isNil(actualOffset) || actualOffset > 0 ? -(radius - innerRadius) * 0.3 : actualOffset;
Copy link
Member Author

Choose a reason for hiding this comment

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

当 offset 没设置或者大于 0 时,默认去 30% 偏移

Copy link
Member

Choose a reason for hiding this comment

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

这个 30% 是经验值吗?

Copy link
Member Author

@visiky visiky Aug 10, 2020

Choose a reason for hiding this comment

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

嗯,先提供经验值,后面我加上备注;还有一些优化的地方

@hustcc hustcc self-requested a review August 10, 2020 06:41
@hustcc hustcc merged commit f3b7fbd into v2 Aug 10, 2020
@hustcc hustcc deleted the enhance-pie branch August 10, 2020 06:41
BBSQQ pushed a commit that referenced this pull request Aug 25, 2020
* feat(v2/core): 增加 default cfg 配置

* feat(v2/legend-adaptor): 增加通用 legend adaptor, 修复 饼图 legend 关闭失效

* feat(v2/pie-inner-label): 增加 pie-inner label 类型 & 增 强 statistic

statistic 增加title 和 content boolean 类型,允许显式关闭

* test: 增加测试用例

* docs(v2/pie): 增加饼/环 demo & api 文档

- 移除 radius 默认配置
BBSQQ pushed a commit that referenced this pull request Aug 25, 2020
* feat(v2/core): 增加 default cfg 配置

* feat(v2/legend-adaptor): 增加通用 legend adaptor, 修复 饼图 legend 关闭失效

* feat(v2/pie-inner-label): 增加 pie-inner label 类型 & 增 强 statistic

statistic 增加title 和 content boolean 类型,允许显式关闭

* test: 增加测试用例

* docs(v2/pie): 增加饼/环 demo & api 文档

- 移除 radius 默认配置
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.

3 participants