Merged
Conversation
- ExpenseChartView: 날짜 범위 선택(드래그/탭) 로직 추가 - SettlementHeaderView: 정산 헤더 UI 리팩터링 - ExpenseListFeature: AsyncStream 로딩 정리, 로컬 캐시 적용 - ExpenseListView: 차트·헤더와 연동된 리스트 UI 개선
- 7일 단위 데이터 분할(dayChunks) - TabView를 이용한 페이징 처리 - 커스텀 페이지 인디케이터 적용
- ExpenseListFeature: currentPage 상태 추가 및 날짜/페이지 기반 필터링 로직 구현 - SettlementHeaderView & ExpenseListView: 상위 뷰에서 차트 페이지 상태 바인딩 연결
- ExpenseListView: 지출 내역 없을 때 empty view 표시 조건 및 메시지 수정 - SettlementResultView: 정산 결과 없을 때 empty view 처리 추가
This file contains hidden or 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
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.
🔗 관련 이슈
✨ 작업 내용
ExpenseChartView에TabView를 적용하여 7일 단위로 지출 내역을 탐색할 수 있도록 변경했습니다.UIPageControl대신 커스텀 인디케이터를 구현하여 차트 하단에 배치하고, 단일 페이지일 경우 숨김 처리했습니다.min 10,000) 및 최소 높이 보정을 적용했습니다.ExpenseListFeature에currentPage상태를 추가하고 필터링 로직을 개선했습니다.expenseEmpty,settlementEmpty)을 추가했습니다.📸 Showcase
📝 참고 사항
Motivation 🥳 (코드를 추가/변경하게 된 이유)
Key Changes 🔥 (주요 구현/변경 사항)
ExpenseChartView:dayChunks를 이용한 데이터 분할 및TabView페이징ExpenseListFeature:formattedTotalAmount계산 시selectedDateRange가nil이면 전체 합계를 반환하도록 수정ExpenseChartView:.chartYScale(domain: 0...max(maxExpense, 10000))적용으로 스케일 왜곡 방지To Reviewers 🙏 (리뷰어에게 전달하고 싶은 말)
ExpenseListFeature의applyFilters와formattedTotalAmount로직을 중점적으로 봐주시면 감사하겠습니다. 차트 페이징과 전체 기간 합계가 의도대로 동작하는지 확인 부탁드립니다.Reference 🔗
Close Issues 🔒 (닫을 Issue)
Close #No.
Checklist