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

MotionPro #240

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

MotionPro #240

wants to merge 25 commits into from

Conversation

bilioicik
Copy link

@bilioicik bilioicik commented May 11, 2024

To avoid unnecessary questions, please see its docs when you review it.

'MotionPro.cmMenuTurnDegreeAround':'绕x:[x]y:[y]旋转[a][d]度并保持[r]距离',
'MotionPro.moveAB':'向[a]移动[b]步',
'MotionPro.moveABX':'向[a]移动[b]步所得x',
'MotionPro.moveABY':'向[a]移动[b]步所得y',
Copy link
Contributor

Choose a reason for hiding this comment

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

由于你积木设计的很特殊,所以我需要你添加一下额外说,你可以在 pr 描述中,补充一下以下内容吗,或者给你的扩展加个 readme

  1. 扩展的设计目标,为了解决什么问题
  2. 扩展积木的数量及他们 分类
  3. 所有扩展积木的截图
  4. 扩展的使用方式效果描述(有视频就更好了)

Copy link
Contributor

Choose a reason for hiding this comment

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

这个积木的设计,把移动和取值操作合并了

整体目的就是为了”少拖动一个积木“吗?为什么要这么做呢?

@fath11
Copy link
Contributor

fath11 commented May 21, 2024

maybe add some separations, other than that everything is good enough for me

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