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

[Home] 후원사 카드 내부 로고 배치 수정 #219

Merged
merged 4 commits into from
Aug 21, 2023

Conversation

koreatlwls
Copy link
Contributor

Issue

Overview (Required)

  • 기존 후원사 카드 내부 로고 배치 수정
  • 자동/무한 스크롤 기능 제거
    • 이 기능은 더 이상 유효하지 않으므로 제거합니다.

Screenshot

@github-actions
Copy link

github-actions bot commented Aug 17, 2023

Test Results

13 tests   13 ✔️  7s ⏱️
  9 suites    0 💤
  9 files      0

Results for commit 7cdb1c7.

♻️ This comment has been updated with latest results.

@laco-dev
Copy link
Contributor

충돌 발생 원인
: 이미지가 Clickable 하도록 변경되었습니다.

end = 24.dp
),
sponsors = goldSponsors
)
Copy link
Contributor

Choose a reason for hiding this comment

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

image

로고들이 가운데를 기준으로 배치되면 좋을 것 같습니다.
*지금은 왼쪽에 쏠려있음

Copy link
Member

Choose a reason for hiding this comment

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

후원사가 4개 고정 배치 기준으로 만들어진 시안이라 SponsorLogo의 size를 fix하지 않고, 부모 영역에 맞춰 늘리는 것이 적절해보입니다. SponsorLogo 컴포저블의 .size(84.dp)를 제거해주세요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

후원사가 4개 고정 배치 기준으로 만들어진 시안이라 SponsorLogo의 size를 fix하지 않고, 부모 영역에 맞춰 늘리는 것이 적절해보입니다. SponsorLogo 컴포저블의 .size(84.dp)를 제거해주세요!

단순히 .size(84.dp) 만 지우게 된다면 화면에서 당근마켓 후원사만 보이게됩니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines 86 to 89
.padding(
start = 69.dp,
end = 24.dp
),
Copy link
Contributor

Choose a reason for hiding this comment

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

만약 데이터가 스크롤이 가능한 정도로 충분히 많다면 오른쪽으로 스크롤을 하게 되면 어떻게 될까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아마 하단의 LazyRow만 scroll한 영역이 왼쪽부분이 줄어든체로 스크롤이 될껍니다.

val scrollState = rememberLazyListState()
val lifecycleOwner = LocalLifecycleOwner.current
val platinumSponsors = sponsors.filter { it.grade == Sponsor.Grade.PLATINUM }
val goldSponsors = sponsors.filter { it.grade == Sponsor.Grade.GOLD }
Copy link
Member

Choose a reason for hiding this comment

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

Recomposition 이 발생하면 이 코드는 어떻게 동작할까요?
적절한 방법을 활용해주세요.
remember 처리

Copy link
Contributor Author

Choose a reason for hiding this comment

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

9e5bbcb
rememberUpdateState 처리하였습니다.

# Conflicts:
#	feature/home/src/main/java/com/droidknights/app2023/feature/home/SponsorCard.kt
Copy link
Contributor

@laco-dev laco-dev left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

다음 작업에 앞서 현재PR은 우선 병합하도록 하겠습니다

@laco-dev laco-dev merged commit 4ef815a into droidknights:main Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Home] 후원사 카드 내부 로고 배치 수정
4 participants