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

Refactor/#732 이미지 불러오는 속도 개선 #738

Merged

Conversation

tmdgh1592
Copy link
Member

@tmdgh1592 tmdgh1592 commented Oct 17, 2023

#️⃣ 연관된 이슈

close : #732

📝 작업 내용

  • 행사, 대회 리사이클러뷰에서 이미지를 미리 불러와서 캐싱하는 기능 구현

스크린샷 (선택)

Before

  • 회색 배경이 오래 보이는 문제
스크린샷 2023-10-18 오전 1 26 53

After

  • 회색 배경이 짧게 보이거나, 아예 보이지 않도록 변경
스크린샷 2023-10-18 오전 1 27 54

비교 GIF

  • PlaceHolder 가 오래 보이는 문제가 해결되었습니다.
    • #8B00FF 2024년 웹개발 트렌드 세미나 화면이 회색(PlaceHolder) 로 보이지 않고 바로 로딩되어 보이는 모습

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 1시간
실제 소요 시간 : 30분

💬 리뷰어 요구사항 (선택)

@tmdgh1592 tmdgh1592 added Android 안드로이드 관련 이슈 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경 High Priority 리뷰 우선순위가 높은 PR Low Priority 리뷰 우선순위가 낮은 PR labels Oct 17, 2023
@tmdgh1592 tmdgh1592 added this to the 6차 스프린트 milestone Oct 17, 2023
@tmdgh1592 tmdgh1592 self-assigned this Oct 17, 2023
Copy link
Collaborator

@chws0508 chws0508 left a comment

Choose a reason for hiding this comment

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

preload라는 기능이 있군요...와우
고생 많으셨습니다!!!
홍실폰으로 실험하면서, 몇장을 preload할지 한번 더 정하면 좋을 것 같습니다

@tmdgh1592
Copy link
Member Author

@chws0508 좋습니다. 이따가 테스트해보죠~

@tmdgh1592 tmdgh1592 merged commit 4a49562 into android-main Oct 18, 2023
1 check passed
@tmdgh1592 tmdgh1592 deleted the Refactor/#732-이미지_불러오는_속도_개선 branch October 18, 2023 06:24
chws0508 pushed a commit that referenced this pull request Oct 18, 2023
* refactor: 오픈 채팅방 URL 설정 Activity를 manifest에서 제거

- 사용되지 않음

* refactor: 컨퍼런스 이미지를 8개까지 미리 불러와 캐싱하도록 변경

* refactor: 대회 이미지를 8개까지 미리 불러와 캐싱하도록 변경
chws0508 added a commit that referenced this pull request Oct 18, 2023
* refactor: toolbar 설정

* refactor: 게시글 글쓰기 화면, appbarLayout -> materialToolbar 하나만 사용하여, 뷰 계층 구조 최소화

* refactor: 함께해요 Detail 화면, appbarLayout -> materialToolbar 하나만 사용하여, 뷰 계층 구조 최소화

* refactor: 함께해요 글쓰기 화면 뷰 계층 최소화

* Refactor/#730 오버드로(over draw) 감소 (#736)

* refactor: 컨퍼런스 화면 Overdraw 최적화

* refactor: 테마의 windowBackground를 white로 변경

* refactor: 테마의 statusBarColor를 투명하게 변경

* refactor: 행사 스크랩 화면 Overdraw 최적화

* refactor: 대회 목록 화면 Overdraw 최적화

* refactor: 프로필 화면 OverDraw 최적화

* refactor: NoContentView OverDraw 최적화

* refactor: 행사 검색 화면OverDraw 최적화

* refactor: 알림 보관함 화면 OverDraw 최적화

* refactor: 행사 상세 화면 OverDraw 최적화

* refactor: 피드 상세 화면 OverDraw 최적화

* refactor: 같이가요 상세 화면 OverDraw 최적화

* refactor: 설정 화면 OverDraw 최적화

* refactor: 알림 설정 화면 OverDraw 최적화

* refactor: 차단 목록 화면 OverDraw 최적화

* refactor: 대댓글 화면 OverDraw 최적화

* refactor: 채팅방 목록 화면 OverDraw 최적화

* refactor: 채팅 목록 화면 OverDraw 최적화

* refactor: 프로필 수정 OverDraw 최적화

* refactor: 프로필 조회 화면 OverDraw 최적화

* refactor: 메인 화면 바텀 네비게이션 배경 투명하게 변경

* Refactor/#732 이미지 불러오는 속도 개선 (#738)

* refactor: 오픈 채팅방 URL 설정 Activity를 manifest에서 제거

- 사용되지 않음

* refactor: 컨퍼런스 이미지를 8개까지 미리 불러와 캐싱하도록 변경

* refactor: 대회 이미지를 8개까지 미리 불러와 캐싱하도록 변경

* refactor: 프로필 수정시, 활동별로 정상적으로 뜨도록 버그 해결 (#744)

* refactor: 함께해요 Detail 화면, appbarLayout -> materialToolbar 하나만 사용하여, 뷰 계층 구조 최소화

* refactor: 테마 전체적용 해제

* refactor: keepScreenOn 제거, 함수명 변경

---------

Co-authored-by: BuNa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR Low Priority 리뷰 우선순위가 낮은 PR 리팩터링 테스트 코드의 검증 값이 변환하지 않고 코드 변경
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants