-
Notifications
You must be signed in to change notification settings - Fork 656
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
フッターのページ下部固定及び、HTMLタグの修正 #5261
フッターのページ下部固定及び、HTMLタグの修正 #5261
Conversation
9bef7e9
to
0e02f2e
Compare
width: 100%; | ||
min-height: 100vh; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
100dvh も一緒に指定するとよいと思います。
https://www.w3.org/TR/css-values-4/#viewport-variants
i(Pad)OS Safari での挙動が求めているものになるはずです。
(vh 指定だと、Safariではアドレスバーなどのサイズが考慮されないので隠れる or 押し出される形になります。)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mogyuchi さん
おっしゃる通りです。ご指摘ありがとうございます。
min-height: 100vh;
min-height: 100dvh;
へ修正したいと思います。
タグが付いていますが、コミットを修正してもよろしいでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@YokoInoue-u
はい、問題ございません。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
min-height: 100dvh;
を追加し、プッシュし直しました。
ご確認よろしくお願い致します。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@YokoInoue-u
修正ありがとうございます。
e2eテストが落ちているようで、みていただくことはできますか?
0e02f2e
to
bd6f193
Compare
Codecov Report
@@ Coverage Diff @@
## 4.1 #5261 +/- ##
============================================
- Coverage 68.57% 68.28% -0.30%
Complexity 6162 6162
============================================
Files 463 463
Lines 25306 25306
============================================
- Hits 17354 17280 -74
- Misses 7952 8026 +74
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This reverts commit f3915f0.
@YokoInoue-u |
@chihiro-adachi |
ご確認お願いいたします。
概要(Overview・Refs Issue)
内容が少ないページ(/help_about/等)でフッターの下に余白ができる現象を修正する。また、ヘッダー等の基本的なHTMLタグを追加する。
方針(Policy)
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目