Fix [#100] : Contracts button gesture - Githubissues

kko-okk / kko_okk

Apple Developer Academy MC2: 꼬옥(kko_okk) (Afternoon session Team16)

GNU Lesser General Public License v2.1

7 stars 1 forks source link

Fix [#100] : Contracts button gesture #151

Closed yeniful closed 2 years ago

yeniful commented 2 years ago

Motivation

셀 동시 제스쳐를 구현해야 했습니다.
스크롤뷰 제스쳐와 셀 제스처가 충돌되는 문제가 있었습니다.

Key Change

id가 같은 셀들을 동시에 누르면 promised로 값이 변하도록 수정했습니다.
Timer를 사용하여 2초정도의 여유 시간을 추가했습니다.
id가 다른 셀들은 함께 눌러도 promised로 값이 변하지 않도록 수정했습니다.

To Reviewers

스크롤뷰에 우선순위를 주기 위한 onTapGesture적용시 longpressgesture의 updating에 있는 애니매이션이 작동되지 않습니다. 방법을 좀 더 찾아보겠습니다. 현재 코드에서는 onTapGesture를 해제해놓았습니다.
promised로 값이 바뀌지는 않지만, id가 다른 셀을 눌렀을 때 애니메이션이 작동되는 버그가 있습니다 asap 해결하겠습니다.
설정 버튼 제스처 밑으로 내려간 이슈도 해결하겠습니다. @seodam-hst

LeeSungNo-ian commented 2 years ago

고생하셨습니다!

yeniful commented 2 years ago

@Valentino1994 MVP 개발 완료 후 조언해주신 것처럼 리팩토링해보겠습니다! 감사합니다!