Skip to content

Feat(effectScatter): doc for effectScatter.clip#181

Merged
pissang merged 2 commits intoapache:masterfrom
susiwen8:clip
Apr 9, 2021
Merged

Feat(effectScatter): doc for effectScatter.clip#181
pissang merged 2 commits intoapache:masterfrom
susiwen8:clip

Conversation

@susiwen8
Copy link
Copy Markdown
Contributor

@susiwen8 susiwen8 commented Apr 6, 2021

Add doc for effectScatter.clip(apache/echarts#14574)

@susiwen8 susiwen8 requested a review from pissang April 6, 2021 12:58
If clip the overflow on the coordinate system. Clip results varies between series:

+ Scatter:Ignore the symbols exceeds the coordinate system. Not clip the elements.
+ EffectScatter the symbols exceeds the coordinate system. Not clip the elements.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It seems there is missing : Ignore.
BTW, may these two series be merged as Scatter/EffectScatter?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

may these two series be merged as Scatter/EffectScatter

Yes, I think that would be better.

@pissang pissang merged commit fcb2b59 into apache:master Apr 9, 2021
@susiwen8 susiwen8 deleted the clip branch April 9, 2021 16:07
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