Skip to content

Commit 6e75b58

Browse files
authored
Homeでユーザー取得時にloading画面にする (#665)
# PRの概要 closes #664 ## 具体的な変更内容 Home画面のユーザー取得時に<FullScreenCircularProgress />が出ないようにし、代わりに<NoMoreUsers />が表示されるようにした。 ## 影響範囲 ## 動作要件 ## 補足 ## レビューリクエストを出す前にチェック! - [ ] 改めてセルフレビューしたか - [ ] 手動での動作検証を行ったか - [ ] server の機能追加ならば、テストを書いたか - 理由: 書いた | server の機能追加ではない - [ ] 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか - 理由: 書いた | 間違った使い方は存在しない - [ ] わかりやすいPRになっているか <!-- レビューリクエスト後は、Slackでもメンションしてお願いすることを推奨します。 -->
1 parent 0b78d47 commit 6e75b58

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

web/app/home/page.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,13 +77,13 @@ export default function Home() {
7777
[recommended, controls, backCardControls],
7878
);
7979

80-
if (recommended == null) {
80+
if (data === undefined) {
8181
return <FullScreenCircularProgress />;
8282
}
8383
if (currentUser == null) {
8484
return <FullScreenCircularProgress />;
8585
}
86-
if (displayedUser == null) {
86+
if (recommended.size() === 0) {
8787
return <NoMoreUser />;
8888
}
8989
if (error) throw error;
@@ -173,4 +173,7 @@ class Queue<T> {
173173
pop(): T | undefined {
174174
return this.store.shift();
175175
}
176+
size(): number {
177+
return this.store.length;
178+
}
176179
}

0 commit comments

Comments
 (0)