Add sound #26
Add sound #26
Conversation
…i-frontend into add_sound
Lighthouse Report
|
Visualizer ReportYou can view the results here. |
…i-frontend into add_sound
src/pages/homePC/index.tsx
Outdated
| <div className={styles["link-copy"]}> | ||
| <TextButton | ||
| text="共有リンクをコピー" | ||
| onClick={handleClick} |
There was a problem hiding this comment.
ここをクリックしてもコピーではなく/yataiに移動している
There was a problem hiding this comment.
ここ、この時点で一旦ルームのuuid含むurlが取得できなかったから一旦この実装にしてました！qrcodeを取り込む時に修正します！🙇！
src/pages/homePC/index.tsx
Outdated
| @@ -0,0 +1,91 @@ | |||
| import { DefaultButton } from "../../components/ui/Button"; | |||
There was a problem hiding this comment.
/pages以下に直接routerから読み込まないJSXを置くのは望ましくない（技術構成によっては予期せぬページが生成されてしまう）のでできれば避けたい。/components/responsiveや/components/commonなどに置くとベターか？Kizukuさんにも聞いた方が良いかも。homeSPも同様。
src/pages/homeSP/index.tsx
Outdated
| <div className={styles["go-game-sp"]}> | ||
| <DefaultButton color="red" size="lg" onClick={handleClick}> | ||
| 射的へ向かう | ||
| </DefaultButton> |
There was a problem hiding this comment.
homePCではホバー中はボタンのサイズが大きくなったが、homePCではホバー中の変化がないのは仕様？
あと、僕の環境ではhomePCをホバーした時にボタンが画面左に少しはみ出していた
There was a problem hiding this comment.
こちらは、いったん、ホバーの方を削除しました、、、
src/pages/homePC/index.tsx
Outdated
| width="200" | ||
| height="200" | ||
| /> | ||
| </div> |
There was a problem hiding this comment.
style.container以下に全てのcomponentを並列に置くのは表示崩れの原因になる（今回は直さなくてもいいから頭の隅にでも）
container
L 動く文字
L 左半分
L ロゴ
L でかいボタン
L 右半分
L QRコード
L QRコード本体
L コピー用ボタン
L 背景画像
のように階層構造でコンポーネントを管理し、一番外側でサイズを調整して内側のコンポーネントはwidth: 100%にすると実装コストも下がる
No description provided.