Skip to content
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

フッターのログインボタン周辺の余白を修正 #997

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

yuw27b
Copy link
Contributor

@yuw27b yuw27b commented Nov 30, 2023

課題・背景

#843 にて、フッターの一番左のカラム(ロゴ・ログインボタンなどのところ)を<section>要素で囲ったため、gridアイテムの適用先が変わってしまい、余白が消えていました。

やったこと

<section>の子要素がgridアイテムになるよう、CSSを調整しました。

動作確認

https://deploy-preview-997--smarthr-design-system.netlify.app/

#843 より前のPRのプレビューを参照し、その時と同じ余白に戻ったことを確認しました。

キャプチャ

Before After

@yuw27b yuw27b self-assigned this Nov 30, 2023
Copy link

netlify bot commented Nov 30, 2023

Deploy Preview for smarthr-design-system ready!

Name Link
🔨 Latest commit 9b7eaee
🔍 Latest deploy log https://app.netlify.com/sites/smarthr-design-system/deploys/656835c91d20030008331a61
😎 Deploy Preview https://deploy-preview-997--smarthr-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@tomof tomof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良さそうです👍

@tomof tomof merged commit 4b0d20c into main Dec 1, 2023
5 checks passed
@tomof tomof deleted the fix/footer-spacing branch December 1, 2023 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants