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

docs: update transform.diffY #6729

Open
wants to merge 1 commit into
base: v5
Choose a base branch
from
Open

docs: update transform.diffY #6729

wants to merge 1 commit into from

Conversation

hustcc
Copy link
Member

@hustcc hustcc commented Apr 1, 2025

Checklist
Description of change

@hustcc hustcc requested a review from interstellarmt April 1, 2025 02:39
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14185882910

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 86.683%

Totals Coverage Status
Change from base Build 14185566640: 0.0%
Covered Lines: 10696
Relevant Lines: 11959

💛 - Coveralls


```ts
import { Chart } from '@antv/g2';
## 开始使用
Copy link
Contributor

Choose a reason for hiding this comment

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

transform部分的文档里大体结构为概述、使用场景、配置项,示例 ,这里开始使用可以修改为使用场景,适当补充相关内容

```

## 选项
## 配置项

| 属性 | 描述 | 类型 | 默认值 |
|-------------------|------------------------------------------------|-------------------------|-----------------------|
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的groupBy其实只能配置通道的名称,感觉需要详细说明下
image

Copy link
Contributor

Choose a reason for hiding this comment

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

有两个可以优化的点

  1. series 的用法比较模糊,哪种情况下为 false 不清晰
  2. 布尔值可以统一去除一下行内代码样式,避免误解为字符串

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants