フッタ作成 #35
Merged
フッタ作成 #35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closed
masayaO
reviewed
Jan 3, 2025
src/components/ui/footer.tsx
Outdated
| const Footer = () => { | ||
| return ( | ||
| <footer className="text-white text-sm"> | ||
| <div className="px-10 py-16 flex flex-col gap-x-16 gap-y-10 max-w-5xl mx-auto md:flex-row md:p-16 lg:gap-x-24"> |
There was a problem hiding this comment.
[Q] max-w-5xlですが、ここはwidth: 100%だと不都合あります？
max-widthで固定すると画像のように白い背景が見えてしまうため確認です。
There was a problem hiding this comment.
こちらは、画面幅が1280px 以上の端末でフッタの中身が左に寄ってしまうことを回避するために設けています。
max-width なし
max-width あり
白い背景が見えてしまうのはまた別の問題であり、こちらのテキストの横幅が絶対値 w-96（= 384px）で指定されているためだと思います。
src/components/ui/footer.tsx
Outdated
Comment on lines
14
to
19
| <ul className="flex flex-col gap-2.5 lg:w-72"> | ||
| <li>ミッション</li> | ||
| <li>プロポーザル募集</li> | ||
| <li>スポンサー一覧</li> | ||
| <li>スタッフ</li> | ||
| </ul> |
There was a problem hiding this comment.
ここですが、下記と同様の理由でスクロールリンクはつけなくて良です！「スポンサー一覧」のみ残しておいてください。ページができたらリンクを設定します
masayaO
approved these changes
Jan 3, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
担当者：梅雨（@tsuyuni ）です
共通コンポーネントとして使用されるフッタの実装になります。
tailwind の config の変更も含みます。
→ 不要であった場合は言っていただければこちらで修正します。
Max-Width の指定が特になかったため、一旦 1024px としてあります。
Noto Sans JP が適用されていません。担当範囲外だと思ったのでそのままにしてあります。
iPhone SE
iPad mini
PC 1280 × 800