[Bug] 버튼 연타 시 중복 호출 이슈 수정 - Githubissues

droidknights / DroidKnightsApp

국내 최대 규모 안드로이드 컨퍼런스 드로이드나이츠 앱

https://www.droidknights.dev

255 stars 69 forks source link

[Bug] 버튼 연타 시 중복 호출 이슈 수정 #243

Closed kisa002 closed 1 year ago

kisa002 commented 1 year ago

Overview (Required)

컴포넌트를 연타할 때 발생하는 중복 호출 이슈를 수정하였습니다.

ISSUE1 뒤로가기 버튼 중복 호출

TopAppBar의 뒤로가기(종료) 버튼을 연타하면 Home 화면을 넘어가 뒤로가는 경우가 발생합니다.
currentDestination이 Home이 아닌 경우에만 뒤로가도록 수정하였습니다.

ISSUE2 화면 중복 이동

홈 화면에서 session 이동 카드를 TopAppBar 영역에 위치한 후 연타하면 화면이 계속 넘어갑니다.
navigate 시 이동할 destination과 currentDestination이 같지 않은 경우에만 이동하도록 함수를 구현하려하였으나
- contributor 화면에서 AppBar 영역을 터치하여도 투과되어 뒤에 있는 카드가 클릭 되기에
- TopAppBar에 빈 clickable을 제공하여 클릭 이벤트가 투과하는 것을 방지하였습니다.

Screenshot

ISSUE1 뒤로가기 버튼 중복 호출

Before	After

ISSUE2 화면 중복 이동

Before	After

github-actions[bot] commented 1 year ago

Test Results

18 tests 18 :heavy_check_mark: 6s :stopwatch: 11 suites   0 :zzz: 11 files   0 :x:

Results for commit 7d98e2b5.

:recycle: This comment has been updated with latest results.

kisa002 commented 1 year ago

남겨주신 리뷰 반영하였습니다! 😁