SoyBeansLab / daizu-online-judge-frontend-old

We will be back. [WIP]
MIT License
4 stars 1 forks source link

paginationのoffsetをredux stateで持つようにした #284

Closed ucpr closed 4 years ago

ucpr commented 4 years ago

概要

対応するIssue

link: #283 #233

netlify[bot] commented 4 years ago

Deploy preview for daizu-front ready!

Built with commit 4ecc88773044812804927ecf6a778d71e0d9ca6c

https://deploy-preview-284--daizu-front.netlify.app

ucpr commented 4 years ago

useEffectのときにinitしないといけないんだけど、それのせいで1回のactionなのに複数回actionが発行されていてうーんとなっている...

ucpr commented 4 years ago

connected-reaxt-routerとかだとシンプルに解決できそう?

ucpr commented 4 years ago

そんなことないなあ...

ucpr commented 4 years ago

というか、paginationを持ったTableはmoleculesではなくorganismsのほうがいい気がする

-> TableをmoleluesでつくってPaginationも持ったTableをorgansismsに置くのがぽい気がする

Organism も比較的簡単で、UI設計、ビジュアルデザイン、HTMLコーディングを実際に行う場合に「〇〇エリア」「〇〇ボックス」「〇〇領域」と呼ばれるもので、それ自体が独立して意味を持ち、ユーザとのインタラクションを定義可能なものという位置付けになる。

https://a-suenami.hatenablog.com/entry/2019/04/29/173415