search

syunto07ka / jewelry_box

2 stars 0 forks source link

pagesディレクトリを切ってルートページを実装する #49

Closed syunto07ka closed 4 years ago

syunto07ka commented 4 years ago

close #45

create branch from #44

syunto07ka commented 4 years ago

@piro0919 修正完了したのでご確認をば〜

piro0919 commented 4 years ago

[nits] ちなみに死ぬほど細かいですが、自分なら

<Route path="/" exact={true}>

ではなく

<Route exact={true} path="/">

と書きます propsをアルファベット順に並べたほうが見た目が良いのでー

syunto07ka commented 4 years ago

Routeというコンポーネントの意味合い的に一番情報として近くにあってほしいものを置くようにと考えていたのですが、そんな個人次第で分かれるような考え方もないですね。それならアルファベットというわかりやすいルールで統一しちゃうのもアリ👀

piro0919 commented 4 years ago

@syunto07ka 余談ですが、わかりやすいルールで統一するのはgoogle的で、意味合いで統一するのはmozilla的だったりします ここらへん調べてみるとむちゃくちゃ面白いですよ 👍