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

✨ feat(curd): 增加展开和收起 #87

Closed
wants to merge 1 commit into from

Conversation

lxc455735702
Copy link
Contributor

可能某些页面搜索内容过多,有些不重要的可以收起来。
之前本来想用navie Grid 格栅组件 里面有个折叠栅格 但是二次封装一直有问题
也考虑过直接定义一套规则去渲染 类似table的columns 但是感觉改造成本有点高 想这样轻量一点
ps:
上传搞了半天 emm 开始用的node 16.15.0 eslint 检查 有报错 eslint 17.9.0 需要高版本的 我换成18.18.0才可以

可能某些页面搜索内容过多,有些不重要的可以收起来。
@lxc455735702
Copy link
Contributor Author

sor
打错字了 crud 和 feat

@zclzone
Copy link
Owner

zclzone commented Jul 10, 2024

@lxc455735702 感谢你的PR,建议不错,但有两点问题需要改进

  1. 不要提交lock文件
  2. 请在原文件修改,复制一个index2.vue文件出来改看不出修改了哪里,没办法review

@lxc455735702
Copy link
Contributor Author

@lxc455735702 感谢你的PR,建议不错,但有两点问题需要改进

  1. 不要提交lock文件
  2. 请在原文件修改,复制一个index2.vue文件出来改看不出修改了哪里,没办法review

收到 一开始我也是原文件修改的 后来还是copy了一份 因为我想让作者先看一下 交换一下意见 不然对原文件改动,可能我的写法不合符规范 sor 之后我调整一下 。

@zclzone
Copy link
Owner

zclzone commented Jul 10, 2024

可能某些页面搜索内容过多,有些不重要的可以收起来。 之前本来想用navie Grid 格栅组件 里面有个折叠栅格 但是二次封装一直有问题 也考虑过直接定义一套规则去渲染 类似table的columns 但是感觉改造成本有点高 想这样轻量一点 ps: 上传搞了半天 emm 开始用的node 16.15.0 eslint 检查 有报错 eslint 17.9.0 需要高版本的 我换成18.18.0才可以

最新版的eslint需要 node 18.18 以上的版本了,我更新下文档

@lxc455735702
Copy link
Contributor Author

可能某些页面搜索内容过多,有些不重要的可以收起来。 之前本来想用navie Grid 格栅组件 里面有个折叠栅格 但是二次封装一直有问题 也考虑过直接定义一套规则去渲染 类似table的columns 但是感觉改造成本有点高 想这样轻量一点 ps: 上传搞了半天 emm 开始用的node 16.15.0 eslint 检查 有报错 eslint 17.9.0 需要高版本的 我换成18.18.0才可以

最新版的eslint需要 node 18.18 以上的版本了,我更新下文档

okk 那我这个先close

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