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

refactor: avoid passing too much stuff in redux #132

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

cheng-kang
Copy link
Collaborator

问题描述

目前由于在 redux 里面存了 dom 元素等导致 redux dev tool 崩溃。这个 PR 的目的是从 redux 中移除这些不必要的内容,仅传输和存储最少的、必要的信息,比如 dom 元素的 id 或者 data-id。

计划

  • 避免在 redux 中传输和存储 iframe 对象
  • 避免在 redux 中传输和存储回调函数:reRect (重命名为 resetRect)
  • 重写获取和更新模板数据的函数,使逻辑更清晰并减少副作用
  • 避免在 redux 中传输和存储当前编辑对象 dom 元素,改为 dom 元素的 id 或 data-id

@jljsj33
Copy link
Member

jljsj33 commented Sep 5, 2019

  • 重写获取和更新模板数据的函数,使逻辑更清晰并减少副作用

这个重写会不会有很多坑,,其实这个我都想去掉,,直接数据放表里,,免得去合并这块数据,,当时是为了结剩点免费表的空间才写的,,你觉得呢??

@cheng-kang
Copy link
Collaborator Author

  • 重写获取和更新模板数据的函数,使逻辑更清晰并减少副作用

这个重写会不会有很多坑,,其实这个我都想去掉,,直接数据放表里,,免得去合并这块数据,,当时是为了结剩点免费表的空间才写的,,你觉得呢??

确实花了很长时间才重写完,我测试了所有用到 setDataSourceValue 和 getDataSourceValue 的地方,应该和之前的一样没什么问题。等这个 PR 完全准备好了,还得麻烦你再测试一下。

直接放表里挺好的,可以记录在 roadmap 里面之后做。我现在主要是在现有的基础上调整一下,不修改任何原本有的行为,方便其他方面的开发和调试。

@jljsj33
Copy link
Member

jljsj33 commented Sep 6, 2019

@cheng-kang 我后面可能在这的维护时间比较少了,,你看可以,你就自已合吧,,先扔上去,,有 bug 再修呗。。。

@jljsj33
Copy link
Member

jljsj33 commented Sep 6, 2019

editor-list 我给你加权限,

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