Skip to content

feat: copy #10

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

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

feat: copy #10

wants to merge 2 commits into from

Conversation

itsatan
Copy link
Contributor

@itsatan itsatan commented Apr 22, 2023

  1. 新增了copy功能
  2. 移除了不必要的yarn.lock文件
  3. 增加了size-limit容量(emmm...10KB 不够用了)

如果可以合入辛苦改一下example图片和README

Copy link
Owner

@jianxiaoBai jianxiaoBai left a comment

Choose a reason for hiding this comment

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

这个功能看起来很挺实用的,感谢贡献,这个 review 能完善一下的话我就 merge 了。

if (Array.isArray(sourceData)) {
sourceData.splice(+key, 0, value)
} else {
Reflect.set(sourceData, key + '副本', Reflect.get(sourceData, key))
Copy link
Owner

@jianxiaoBai jianxiaoBai Apr 24, 2023

Choose a reason for hiding this comment

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

  1. 这里把 "副本" 改成 "copy" 吧
  2. 这还有个小问题就是只能新增一个,不能连续新增,新增时 key 的副本也会进行自增,如 key copy 1/ key copy

Copy link
Contributor Author

Choose a reason for hiding this comment

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

由于field文本框宽度为100px,此处修改为key copy 1 - key copy n 会造成key过长不方便阅读,所以我改为了key-n这种类型,去掉了copy,并且复制出来的数据禁止了的copy功能

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.

2 participants