[홈] 데일리 문제 리스트 화면 구현 #94
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
기존에 시간 상수값 넣어서 계산하던 방식 개선 - 시간이 지날수록 애니메이션과 icon 갱신 시간이 달라지는 이슈 발생 - 애니메이션 시점을 Store 받아 icon 업데이트하는 방식으로 수정
- 시작 화면 - 데일리 리스트 화면
|
엥. 데일리 문제 푼 후 상단 타이틀이 다르네. 수정해둘게요 ~! |
enebin
reviewed
Sep 7, 2023
Comment on lines
53
to
60
| return .run { send in | ||
| repeat { | ||
| for icon in icons { | ||
| await send(.setIcon(icon)) | ||
| try await self.clock.sleep(for: .seconds(1.595)) | ||
| await send(.toggleAnimation(icon)) | ||
| try await self.clock.sleep(for: .seconds(0.85)) | ||
| } | ||
| } while true | ||
| } |
There was a problem hiding this comment.
Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-09-07.at.14.50.43.mp4
요 부분이 StartView init 마다 중첩돼서 탭 전환할 때마다 위에 캡처처럼 애니메이션이 깨지는 것 같음ㅋㅋ
제 생각엔 while true를 조절할 수 있게 만들거나 withTaskCancellation 써서 실행중이면 취소시키고 새걸로 돌리는 식으로 하면 해결 될 듯?
There was a problem hiding this comment.
이거 좀 확인한 결과, 마이페이지 탭 넘어가도 홈 탭의 Feature가 죽지 않아 생기는 이슈였음 . .
그래서 홈의 뷰들에서 onDisappear action 추가해서 해당 뷰 사라질 때, 실행중인 task들을 cancel 해줬어 !
다만, 이게 진행중이던 icon이 이어서 진행되는데 괜찮을까 ?
Simulator.Screen.Recording.-.iPhone.13.mini.-.2023-09-07.at.16.51.03.mp4
There was a problem hiding this comment.
그거 + 혹시 모를 케이스 대비에서 애니메이션 시작 전에
guard isAnimating == false else { return } 하나 걸어주셔도 괜찮을 듯?
There was a problem hiding this comment.
이거 적용하면 애니메이션 적용이 잘 안돼서 . . 추후에 수정 다시 할게요 !
|
와 너무 잘해주셨음!!!! 코멘트 단 거 하나만 확인해주면 될 듯ㅎㅎ |
각 View Disappear 되면 Task cancel해서 해결
서버값 받아온 후 뷰 그리도록 수정
0inn
force-pushed
the
feat/home-daily-tests(#93)
branch
from
c77a6b5
to
5ef0033
Compare
|
|
굿입니다. 일단 머지 할게요~ |
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.
closed: #73
closed: #93
홈 애니메이션 이슈 해결
애니메이션 시간을 상수로 넣어놔서 몇 번 반복하면, 애니메이션과 아이콘이 정확하게 매칭 안되는 이슈
-> repeatforever() 대신, TCA 구조를 사용해 애니메이션 토글 구현
시작하기 / 데일리 분기
로 확인해서 분기처리 해뒀습니다.
남은 작업