Refactor/#730 오버드로(over draw) 감소 #736
Merged
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
tmdgh1592
added
Android
chws0508
approved these changes
Oct 18, 2023
chws0508
pushed a commit
that referenced
this pull request
Oct 18, 2023
* 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: 메인 화면 바텀 네비게이션 배경 투명하게 변경
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
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.
#️⃣ 연관된 이슈
📝 작업 내용
불필요하게 background가 지정되어 있어서 OverDraw되는 현상을 방지하였습니다.
웹사이트로 이동버튼 부분은 쉐도우가 적용되어 있어서 어쩔 수 없이 해결하지 못했습니다.그 외의 모든 뷰는 최대한 OverDraw가 발생하지 않거나, 적게 발생하도록 리팩터링 완료하였습니다.
스크린샷 (선택)
모든 View에 OverDraw 최적화를 진행하였으나,
예시로 EventFragment 화면을 첨부하였습니다.
Before
After
예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)
예상 소요 시간 : 6시간
실제 소요 시간 : 6시간
💬 리뷰어 요구사항 (선택)
코드는 Background 제거 외에 건든 부분은 거의 없습니다.
따라서 코드보다는, GPU OverDraw 디버깅 모드를 통해 확인만 해주시면 좋겠습니다.