Skip to content
This repository has been archived by the owner on Apr 11, 2022. It is now read-only.

Vue 组件 - 把 computed 和 deepClone 给封装到 <Editor> 组件内部,不要让用户配置 #382

Open
wangfupeng1988 opened this issue Feb 7, 2022 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@wangfupeng1988
Copy link
Contributor

如微信沟通。

注意,要兼容当前用户自己写 computed 和 deepClone 的情况

改完先不要发布,把 pr 贴到这里,我本地测试一下。

@wangfupeng1988 wangfupeng1988 added the enhancement New feature or request label Feb 7, 2022
@itshizhan
Copy link

可以使用v-model 即可

@wangfupeng1988
Copy link
Contributor Author

可以使用v-model 即可

暂时先不会使用 v-model ,因为编辑器内部是自渲染的,并不是 vue 渲染的。

@iCloudys
Copy link

v-model 和渲染什么关系?

@clinfc
Copy link
Contributor

clinfc commented Feb 19, 2022

v-model 和渲染什么关系?

要考虑响应式 API(reactive/ref) 对数据的污染问题,响应式 API 造成的污染会直接让编辑器崩溃。deepClone 就是规避污染的方式之一。

@wangfupeng1988
Copy link
Contributor Author

@wangfupeng1988
Copy link
Contributor Author

另外注意,如果只用 defaultHtml 而不用 defaultContent ,是不需要 deepClone 的。
这一点需要兼容

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants