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] 배경색아 제발 #466

Merged
merged 7 commits into from
Aug 22, 2024
Merged

[FE] 배경색아 제발 #466

merged 7 commits into from
Aug 22, 2024

Conversation

jinhokim98
Copy link
Contributor

@jinhokim98 jinhokim98 commented Aug 21, 2024

issue

구현 사항

  • 스크롤이 길어질 때 배경색이 짤리고 길어지지 않았던 버그 수정했어요
    • #root의 속성에서 height: 100%을 제거했어요.
    • 스크롤이 생기지 않을 정도로 내용이 부족할 때 메인 레이아웃의 배경색이 안 채워지는 문제가 있었습니다. 그래서 MainLayout에 min height: 100vh를 추가로 넣으니 내용이 부족할 때도 배경색이 화면을 꽉 채우고 스크롤이 생겨도 같이 배경색이 따라서 내려오게 됩니다.
    • 이도 행동 디자인이 변경됐기 때문에 행동 디자인 재배포가 필요합니다.
2024-08-22-19-50-30.mp4
  • 메인 레이아웃 배경색 변경

    • 토다리와 배경색 논의 결과 메인 레이아웃의 배경색을 gray에서 lightGray로 바꾸자는 결론이 나서 메인레이아웃 backgroundColor props로 lightGray도 추가했습니다.
    • 적용하기 위해서 행동 디자인 재배포가 필요하며, 배포가 된 후에 적용을 해야합니다.
  • 생성 페이지 배경색 하얀색으로 설정

    • 메인 레이아웃 background color props를 white를 적용했어요
  • 이벤트 레이아웃 padding bottom 조정

    • home의 경우 아래 아무 패딩도 없어서 2rem을 띄웠습니다
    • admin의 경우 fixed button 때문에 8rem이 띄워졌는데 사라졌기 때문에 home과 동일하게 2rem으로 설정했습니다.

🫡 참고사항

배경색 너무 힘들어요;;

@jinhokim98 jinhokim98 added this to the lev3 milestone Aug 21, 2024
@jinhokim98 jinhokim98 requested review from pakxe, soi-ha and Todari August 21, 2024 14:09
@jinhokim98 jinhokim98 self-assigned this Aug 21, 2024
@jinhokim98 jinhokim98 linked an issue Aug 21, 2024 that may be closed by this pull request
Copy link
Contributor

@soi-ha soi-ha left a comment

Choose a reason for hiding this comment

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

몇번째 고통받고 있는지...흙흑

Copy link
Contributor

@Todari Todari left a comment

Choose a reason for hiding this comment

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

진짜 배경색아 제발... 우리를 괴롭히지마...

Copy link

Copy link

@pakxe pakxe merged commit dcf0387 into fe-dev Aug 22, 2024
2 checks passed
@pakxe pakxe deleted the feature/#464 branch August 22, 2024 15:01
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.

[FE] 배경색아 제발
4 participants