Meituan-Dianping / beeshell

React Native 组件库
MIT License
1.8k stars 280 forks source link

TopView的设计很有问题 #85

Closed s949492225 closed 4 years ago

s949492225 commented 4 years ago

1.TopView作为浮层的管理者,怎么可以把自身放入浮层的栈中呢. 2.使用者实际的dom应该不会因为浮层栈的变动而更新,请参考react-navigation的设计,把真实的dom用pureComponent包裹起来 3.每一个浮层都有跟2一样的问题,每一个浮层都应该是独立的,不受外部变化而受影响 4.TopView缺失更新某个浮层的功能,很多情况下浮层是需要更新的,比如Toast这种场景,连续弹出了两个Toast但是内容不同,这时候是需要更新的Toast组件的内容的 5.replace是否有必要,添加,更新,移出三个主体功能其实个人感觉就已经足够了. 6.hook的形式有非常大的风险,一旦多个库都适用同样的方式hook,会导致很大的问题,建议谨慎保留 LayerXXX.js.zip

给你一个跟TopView差不错的的东西参考下,这个设计参考了flutter的实现