현위치에서 키워드로 재검색, 검색 시 필터 초기화 #89
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
jeeminimini
reviewed
Feb 19, 2024
| ) { | ||
| val mapScreenType by mapViewModel.mapScreenType.collectAsStateWithLifecycle() | ||
| val context = LocalContext.current | ||
| var backPressedTime = 0L | ||
|
|
||
| BackHandler { | ||
| if (mapScreenType == MapScreenType.SEARCH) { | ||
| mapViewModel.updateIsSearchTerminated(true) |
There was a problem hiding this comment.
MapScreenType.SEARCH에서 BackHandler를 통해 SearchScreen으로 갔다가 다른 키워드를 검색해 MapScreenType.SEARCH으로 넘어가는 경우, 필터 초기화를 해주기 위함입니다! 만약 이 때 필터 초기화가 안 된다면 새로운 키워드에 대한 결과 마커가 필터로 모두 가려졌을 때 사용자에게 혼란을 줄 수 있다고 생각했습니다.
jeeminimini
reviewed
Feb 19, 2024
Comment on lines
181
to
189
| ReSearchComponent( | ||
| isMarkerClicked, | ||
| currentSummaryInfoHeight, | ||
| isMapGestured, | ||
| onReSearchButtonChanged, | ||
| onMarkerChanged, | ||
| onBottomSheetChanged, | ||
| isLoading, | ||
| ) |
There was a problem hiding this comment.
이 친구 이름도 Button으로 하면 좋을 것 같아요!
- 불필요한 SearchText 삭제 - ReSearchButtonComponent로 이름 변경
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.
⭐️ Issue Number
🚩 Summary
앱 터지는 문제 해결!!
원인 : BasicTextField에서 innerTextField는 한 번만 호출되어야하는데, if문 안에서 여러번 호출될 때마다 앱이 터졌던 것
🛠️ Technical Concerns
최근 검색어 리스트 클릭 시, 해당 키워드에 대해 검색을 진행하고 지도로 이동한 뒤 결과를 띄워주는 과정이 SearchScreen의 BasicTextField에서 keyboardActions가 onDone일 때의 로직과 같다고 판단해 같은 코드를 넣어주었습니다. (if (searchText.isNotBlank()) 부분 제외)
그런데 마커와 바텀시트에 검색 결과 데이터가 아닌, 가게 상세정보 데이터가 노출되는 버그가 발생했습니다.
NaverMapScreen에서
if (mapScreenType == MapScreenType.MAIN)쪽으로 넘어가서 생기는 문제인 것 같은데 RecentSearchItem내부에서 MapScreenType.SEARCH로 업데이트 해줬는데도 이런 문제가 발생하는 원인이 뭘까요?ㅜㅜ🙂 To Reviwer
📋 To Do