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

[FE] 배경색상 height 버그 #443

Merged
merged 2 commits into from
Aug 21, 2024
Merged

[FE] 배경색상 height 버그 #443

merged 2 commits into from
Aug 21, 2024

Conversation

soi-ha
Copy link
Contributor

@soi-ha soi-ha commented Aug 21, 2024

issue

구현 사항

몇번째인지 모르겠는... heigth 잘림 수정..

간단하게 일단 문제를 해결하고자 html 태그에 background-color를 넣어줬어요.
color를 넣어줌으로 배경색이 더이상 잘리지 않습니다! 왜냐면 기본 틀의 색상을 그냥 저걸로 박어버렸기에..

스크린샷 2024-08-21 16 46 32 스크린샷 2024-08-21 16 46 23

하지만 문제는 랜딩페이지입니다.
html의 배경색이 더이상 화이트가 아니기 때문에.. 다음과 같이 배경색이 들어가게 됩니다.
근데 일단은 랜딩페이지에 배경색이 들어가는 것 보다는 관리/홈 페이지에서 배경색이 잘리는 것이 더 크리티컬하다고 판단하여 일단 해당 방식으로 해결했습니다.

스크린샷 2024-08-21 16 46 43

사실 더 괜찮게 해결할 수 있는 방법이 존재합니다.
id가 root인 태그의 바로 아래 태그에는 height가 100% 존재합니다. 이 height를 삭제해줍니다.
그러면 홈/관리 Nav 탭이 사라지게 됩니다. 이유는 해당 Nav 탭에는 height가 존재하지 않기 때문입니다.
따라서 홈/관리 Nav 탭에 heigth를 지정해주면 됩니다. (개발자도구에서만 확인했던 사항이라 정확하진 않을 것입니다.)

하지만 해당 방법은 수정을 하려면 디자인시스템도 수정해야 합니다. 그리고 공수도 더 들어가구요. 일단 당장 금요일이 시연이기 때문에 더 간단한 방식으로 해결했습니다.

🫡 참고사항

@soi-ha soi-ha added 🖥️ FE Frontend 🚨 bug bug labels Aug 21, 2024
@soi-ha soi-ha added this to the lev3 milestone Aug 21, 2024
@soi-ha soi-ha self-assigned this Aug 21, 2024
@pakxe pakxe self-requested a review August 21, 2024 07:56
Copy link
Contributor

@jinhokim98 jinhokim98 left a comment

Choose a reason for hiding this comment

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

스머

@jinhokim98 jinhokim98 merged commit a8f0af4 into fe-dev Aug 21, 2024
1 check passed
@jinhokim98 jinhokim98 deleted the feature/#440 branch August 21, 2024 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants