-
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
[feat] Artworks(작품목록) 화면 구성 #15 #26
Open
easyhooon
wants to merge
12
commits into
develop
Choose a base branch
from
feat/#15
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+473
−43
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
landscapist-coil 라이브러리 적용
- 이미지 로드 실패시, 실패 이미지가 두개가 다른 비율로 로드되는 문제 해결 - 이미지가 사진의 비율에 따라 세로 크기가 변동되도록 구현(가로 크기 고정) - 작품목록 리스트 상하 padding 적용
2 tasks
전역적으로 설정한 이미지 캐싱 전략과 NetworkImage 내에 ImageLoader 의 캐싱 전략이 충돌되어 발생했다고 판단되어짐
Kotlin 2.0 이후 버전으로 kotlin 과 compose compiler 가 버전 통합되어 별도로 compose 라이브러리들의 버전을 관리해줄 필요가 없어짐
초기에 placeholder 이미지의 크기만큼 이미지의 영역이 할당되는 문제 해결
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 관련 이슈
📁 작업 설명
📸 작업화면(선택)
제가 이해한 화면내 작품 아이템의 가로, 세로 비율은 아래 영상과 같은데, 맞는 방향인지 확인 부탁드립니다.
이미지(png)
png_placeholder.mp4
이미지(Url)
https://placehold.co/
dynamic_placeholder.mp4
레퍼런스
Jetpack Compose Coil 이미지 캐싱, 잘 하고 계신가요?
Jetpack Compose Coil 캐싱, 어떻게 하고 있을까요?- 디스크 캐싱
문제
dynamic placeholder(imageUrl 을 통한 이미지 로드)의 경우, 아래 방향으로 스크롤 할땐 문제가 없으나 위로 스크롤시 스크롤이 스무스하게 진행되지 않는 문제가 있어, 여러 방법을 시도해봤는데 문제가 해결된것같지 않아 원인을 좀 더 찾아봐야할 것 같습니다.
이후 작업 진행하면서 문제 해결해보도록 하겠습니다.해결
8eb0398
6f9545c
간헐적으로 스크롤이 스킵되는 현상 발생
landscapist.mp4
간헐적으로 바운드 되는 현상 발생
AsyncImage.mp4