fix: 캘린더 인터랙션 개선#329
Conversation
|
@coderabbitai summary |
jihun32
left a comment
jihun32
left a comment
There was a problem hiding this comment.
고생해써 👍
캘린더 높이가 고정되어 있어서 날짜가 적은 달은 아래 빈 영역이 너무 크게 나오는데 이거 고쳐주고 코멘트 하나 확인해줘~
| .transaction { transaction in | ||
| transaction.animation = nil | ||
| } | ||
|
|
||
| if UITestMode.isProbeScenario { | ||
| calendarView.perfStateMarker( | ||
| slug: "home", | ||
| key: "calendar-month", | ||
| value: "\(store.calendarDate.year)-\(store.calendarDate.month)" | ||
| ) | ||
| .transaction { transaction in | ||
| transaction.animation = nil | ||
| } |
There was a problem hiding this comment.
이거 내가 전에 적용했던건데 TestMode에만 적용하면 주간 캘린더 인터랙션이 약간 룰렛돌아가는 것처럼 되더라고?
#319 여기 영상이랑 비교해보면 좀 느껴질텐데 나는 319 영상이 더 자연스럽게 느껴지는데 확인해보고 의견 줘!
(목표 편집 리스트에 주간 캘린더 인터랙션이랑 비교해보면 될듯)
|
@jihun32 캘린더 높이 같은 경우에는... 여러 시도해보다가 고정하기로 했던게, swipe를 빠르게 휙휙 하면서 애니메이션 테스트를 했는데 (이 떄가 가장 많이 끊기는 상황이라서) 이 경우에 body update 비용이 꽤 비싸서 높이까지 가변적으로 가면 조금 어렵더라고... 그래서 일부러 냅뒀는데 조금 별로인가?? 흠... 디자이너들이랑 이야기 해봐야겠다 |
ㅇㅋㅇㅋ 디자이너랑 얘기해보고 정해진 결과대로 하자 |
|
@jihun32 디자이너는 지금 애니메이션이 맞다고 보는듯!! 그리고 홈화면 플레이스홀더 관련해서 언급 있었는데, 간단한거니 코멘트 준거 개선할때 같이 하고 머지할게 |
clxxrlove
force-pushed
the
fix/#328/TWI-112
branch
from
13f9924 to
2f8e769
Compare
2 participants
🔗 관련 이슈
📙 작업 내역
🎨 스크린샷 또는 시연 영상 (선택)
ScreenRecording_05-30-2026.16-51-00_1.MP4