akiran / akiran/react-foundation-apps

Modal and layers

オープン
#4 コメント 4 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
291
フォーク
45
PR マージ指標
30日以内にマージされた PR はありません

説明

First, thanks for the library! Good work!

Looking at your code, it seems that you implement modal dialogs by rendering them in the DOM and showing or hiding them. This does seem to be the Angular / Foundation for Apps way.

However, in React modals should use layers to render them on demand.

See:

https://github.com/facebook/react/issues/379

and (the solution)

https://github.com/pieterv/react-layers

What do you think?

Peter

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。