-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FEATURE] #33-Product Detail UI 구현 #40
Conversation
bb917fa
to
e0d43e0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생 많으셨습니다~~!!👍👍
궁금한 부분 코멘트 남겼습니다! 확인 부탁드릴게요!🙇♂️🙇♂️
item.eventList.forEachIndexed { index, event -> | ||
val eventView = ListItemEventBinding.inflate(LayoutInflater.from(binding.root.context), binding.root, false).apply { | ||
tvEventDes.text = event.title | ||
ivStoreIcon.load(item.imageUrl) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://coil-kt.github.io/coil/getting_started/
coil 라이브러리 사용시 Application class에서 ImageLoaderFactory
를 구현 안해줘도 괜찮을까요~?.?
load()
확장함수는 내부적으로
val imageLoader = imageView.context.imageLoader
val request = ImageRequest.Builder(imageView.context)
.data("https://example.com/image.jpg")
.target(imageView)
.build()
imageLoader.enqueue(request)
위 코드를 실행하는데, ImageLoaderFactory
구현을 안하면 ImageLoader는 기본적으로 lazy하게 생성되는 것으로 알고있습니다!
제가 잘못 알고 있는 부분이 있다면 코멘트 남겨주시면 감사하겠습니답!!🙇🙇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lazy하게 생성되어도 괜찮을 것 같아요~!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 리뷰가 늦어져 죄송해요!🙏
덕분에 많이 배워 갑니다. 👍🏻
tvThumbsUpPercent.text = "${item.upvoteRate}%" | ||
tvThumbsDownPercent.text = "${item.downvoteRate}%" | ||
|
||
vThumbsUpRate.updateLayoutParams<LinearLayout.LayoutParams> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오. 이 방법 배워갑니다. 👍🏻
CustomView만 생각하고 있었는데, 이런 방법이 있었네요.
figma : https://www.figma.com/file/NREeyZFNDZGUrPb0JUfIoX/%ED%8E%B8%EB%A6%AC-Design?node-id=364%3A2123&mode=dev