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.

Sign up for GitHub

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

Jump to bottom

feat(word-cloud): 支持自定义random #1791

Merged
merged 10 commits into from
Oct 27, 2020
Merged

feat(word-cloud): 支持自定义random #1791

merged 10 commits into from
Oct 27, 2020

Conversation

zhangzhonghe
Copy link
Member

@zhangzhonghe zhangzhonghe commented Oct 26, 2020 •
edited
Loading

close #1755

  • 新增 random 配置项，用以更改算法内部使用的随机方法，如果设置成固定的值，则可以实现每次渲染的单词布局一致。
  • 新增 spiral 配置项。
  • 去除 rotateRatio 选项，该选项没有必要存在。
  • rotation 支持传入 number 和 函数，用以支持更精确的控制旋转角度。
  • 新增 placementStrategy 选项，用以自定义每个单词的布局坐标。
  • 更新单测。
  • 更新文档。
  • 添加demo。

@zhangzhonghe zhangzhonghe requested review from hustcc and lxfu1 October 26, 2020 08:24
@github-actions
Copy link
Contributor

github-actions bot commented Oct 26, 2020 •
edited
Loading

😭 Deploy PR Preview d9af79d failed. Build logs

🤖 By surge-preview

@hustcc
Copy link
Member

hustcc commented Oct 26, 2020

单测报错。

@zhangzhonghe
Copy link
Member Author

单测报错。

是的，之前的报错已经改好了

@zhangzhonghe zhangzhonghe marked this pull request as draft October 26, 2020 14:01
@zhangzhonghe zhangzhonghe marked this pull request as ready for review October 27, 2020 07:09
@hustcc hustcc merged commit d9af79d into master Oct 27, 2020
@hustcc hustcc deleted the feat-issue-1755 branch October 27, 2020 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hustcc hustcc hustcc approved these changes

@lxfu1 lxfu1 Awaiting requested review from lxfu1

Assignees
No one assigned
Labels
PR: merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wordcloud custom placement strategies
2 participants
@zhangzhonghe @hustcc