Skip to content

refactor: avoid passing too much stuff in redux#132

Draft
cheng-kang wants to merge 4 commits intomasterfrom
feat-tree-select
Draft

refactor: avoid passing too much stuff in redux#132
cheng-kang wants to merge 4 commits intomasterfrom
feat-tree-select

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

2 participants