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

优化 label 类以及修复相关 BUG #2143

Merged
merged 16 commits into from
Mar 20, 2020
Merged

优化 label 类以及修复相关 BUG #2143

merged 16 commits into from
Mar 20, 2020

Conversation

simaQ
Copy link
Contributor

@simaQ simaQ commented Mar 10, 2020

  • fix: Interval label 对于转置坐标系下 position 的位置计算错误

  • refactor: Pie label 的布局调整从 PieGeometryLabel 类逻辑中抽离,使用 label 布局函数实现

  • refactor: Label 基类添加 layout 默认布局
    将 LabelRender 的生成移至 GeometryLabel 的 render 方法中,由 GeometryLabel 统一负责生成 label 信息以及渲染图形。

  • 修复漏斗图相关 bug:
    * 更新时 label 位置错误
    * 尖底漏斗图最后一个图形 label 定位问题 Closed 尖底漏斗图, 最后一个label位置计算错误 #1847

  • 测试用例修改 + 补充

@lgtm-com
Copy link

lgtm-com bot commented Mar 11, 2020

This pull request introduces 4 alerts when merging e03dcc0 into d1b7606 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Mar 16, 2020

This pull request introduces 2 alerts when merging f980668 into fdf485c - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@simaQ simaQ mentioned this pull request Mar 17, 2020
3 tasks
@lgtm-com
Copy link

lgtm-com bot commented Mar 17, 2020

This pull request introduces 3 alerts when merging 9c7f2a4 into fdf485c - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@simaQ simaQ removed the WIP label Mar 17, 2020
@simaQ simaQ changed the title WIP: 优化 label 类 优化 label 类以及修复相关 BUG Mar 17, 2020
@simaQ simaQ requested a review from hustcc March 17, 2020 05:26
@simaQ simaQ added this to the v4.0.3 milestone Mar 17, 2020
@lgtm-com
Copy link

lgtm-com bot commented Mar 18, 2020

This pull request introduces 4 alerts when merging 5c43872 into 4b97895 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Mar 18, 2020

This pull request introduces 4 alerts when merging 8e811a4 into 4b97895 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

src/component/labels.ts Show resolved Hide resolved
src/component/labels.ts Outdated Show resolved Hide resolved
src/chart/view.ts Outdated Show resolved Hide resolved
@simaQ simaQ force-pushed the label-refactor branch 2 times, most recently from b2b8cca to 0d4d582 Compare March 19, 2020 03:06
@lgtm-com
Copy link

lgtm-com bot commented Mar 20, 2020

This pull request introduces 3 alerts when merging ac4f3f4 into 394d8e8 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@visiky visiky self-assigned this Mar 20, 2020
@simaQ simaQ requested a review from visiky March 20, 2020 06:20
@visiky visiky merged commit cfe9e3f into master Mar 20, 2020
@visiky visiky deleted the label-refactor branch March 20, 2020 06:21
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.

尖底漏斗图, 最后一个label位置计算错误
2 participants