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

[ISSUE-81] 홈 화면 UI 수정 및 컴포즈, 코틀린 버전 업데이트 #87

Merged
merged 6 commits into from
Jul 12, 2022

Conversation

hoyahozz
Copy link
Member

@hoyahozz hoyahozz commented Jul 10, 2022

ISSUE


작업 내용

  • 상단 박스의 상단 그림자가 잘리지 않도록 수정
  • HomeOutlinedButton을 없애고, Icon으로 설정, Clickable 로 클릭 이벤트 구현
  • 코틀린, 컴포즈 버전을 상향
    • 상향함에 따라 SuppressLint 가 필요한 부분은 임의로 추가하였습니다.
    • 그런데 알아보니 큰 공수가 드는 작업은 아니라서, 괜찮으시면 제가 SuppressLint 걷어내고 일부분 수정해도 될 것 같습니당~
    • References : https://stackoverflow.com/questions/72084865/content-padding-parameter-it-is-not-used
      • 요약 : 레이아웃 문제를 해결하기 위함, scaffoldpadding 을 추가해주어야 레이아웃 아이템을 가리는 문제가 생기지 않기 때문에, 컴포즈에서는 필수적으로 넣을 것을 권장하고 있음.
      • (저번에 PlanzScreeninnerPadding 을 넣어줬던 이유와 일맥상통합니다~!)

실행 화면

Screen Shot
Screenshot_1657441715

Check List

  • PR 제목은 [ISSUE-{N}] PR 제목으로 작성
  • CI/CD 통과 여부
  • 1명 이상의 Approve 확인 후 Merge
  • 관련 이슈 연결

@hoyahozz hoyahozz self-assigned this Jul 10, 2022
@hoyahozz hoyahozz added this to the Sprint #6 milestone Jul 10, 2022
Copy link
Member

@jihee-dev jihee-dev left a comment

Choose a reason for hiding this comment

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

좋아용 고생하셨습니다!!

@hoyahozz hoyahozz merged commit 2bd6ee4 into develop Jul 12, 2022
@hoyahozz hoyahozz deleted the fix/issue-081-home-ui branch July 12, 2022 09:10
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.

[FIX] 홈 화면 UI 일부 수정
2 participants