-
Notifications
You must be signed in to change notification settings - Fork 492
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
Toast 组件用方法调用似乎不符合 React 哲学 #7
Comments
欢迎参与探讨,使用属性控制确实可以通过 |
可以只稍微改一下现有的实现, 让 |
我想就是尽量不需要单独用到组件方法 能不要state就不要 要state的话能用属性控制就用属性控制。因为比如说在loading的话就是上级组件的一个state loading来决定就好了 |
同意 @n7best , React 的理念是单向数据流, 组件会随着数据的变化局部刷新. 我之前提到的修改方案仔细想一下并不合适, 因为 |
组件简单的话,组件内维护一个 |
#7 remove toast `show` & hide method, use `show` prop instead
#7 remove dialog `show` & `hide` method, use `show` prop instead
首先赞一下这个库。 赞同用 属性show的方式来控制 组件的显示。 看目前代码已经修改, 不过这份 文档 还未修改~ |
囧,toast的说明文档改了,上手文档的示例没改,我改一下,谢谢提醒~ |
#7 remove ActionSheet `show` & hide method, use `show` prop instead, …
nice,最近正好用得着Loading的Toast |
之前 n7 的设计是用
show
属性控制显示, 个人更喜欢之前的设计.用属性控制显示的好处是属性可以接受
state
或其他store
传递来的数据.The text was updated successfully, but these errors were encountered: