Skip to content

Conversation

@DongJun-H
Copy link
Member

작업목표

  • 검색 화면및 검색결과 화면에서 기기의 뒤로가기 버튼과, 상단 액션바 좌측 뒤로가기 아이콘 버튼이 동일한 동작이 하지 않는 현상에 대한 해결

작업사항

  • BackHandler를 통해, 기존 아이콘버튼에 대한 동작을 적용했던것과 동일하게 적용

참고

@DongJun-H DongJun-H requested review from Copilot and yuni-ju May 25, 2025 07:49
@DongJun-H DongJun-H self-assigned this May 25, 2025
@DongJun-H DongJun-H added the bug Something isn't working label May 25, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR ensures the device back button and the action bar back icon perform the same navigation in the search screens.

  • Adds BackHandler to SearchRoute for consistent back behavior.
  • Adds BackHandler to SearchResultRoute for consistent back behavior.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
SearchScreen.kt Imported BackHandler and wired onBackClick() to it
SearchResultScreen.kt Imported BackHandler and wired onBackClick() to it
Comments suppressed due to low confidence (2)

presentation/src/main/java/daily/dayo/presentation/screen/search/SearchScreen.kt:72

  • Consider adding a UI or integration test to verify that both the hardware back button and the action bar back icon invoke onBackClick() in this screen.
BackHandler {

presentation/src/main/java/daily/dayo/presentation/screen/search/SearchResultScreen.kt:109

  • Consider adding a UI or integration test to verify that both the hardware back button and the action bar back icon invoke onBackClick() in this screen.
BackHandler {

Copy link
Member

@yuni-ju yuni-ju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May-28-2025.22-22-05.mp4
  1. 최근검색어에서 키워드를 클릭했을때는 검색화면에서 바로 뒤로가기가 되는데, 직접 검색어를 입력한 후 뒤로가기를 누르면 두번 눌러야 뒤로가지는 현상이 발생합니다.

  2. 이건 이번 이슈랑은 관계 없지만... 최근 검색어에 test말고 아래 한글로된 검색어가 가운데 정렬이 안되어있는 것 --> 제가 습관적으로 엔터 버튼을 눌렀더니 검색어에 줄바꿈 기호가 포함되는 것 같더라구요! 요것도 추후 제거 되면 좋을 것 같아요

@DongJun-H
Copy link
Member Author

May-28-2025.22-22-05.mp4

  1. 최근검색어에서 키워드를 클릭했을때는 검색화면에서 바로 뒤로가기가 되는데, 직접 검색어를 입력한 후 뒤로가기를 누르면 두번 눌러야 뒤로가지는 현상이 발생합니다.
  2. 이건 이번 이슈랑은 관계 없지만... 최근 검색어에 test말고 아래 한글로된 검색어가 가운데 정렬이 안되어있는 것 --> 제가 습관적으로 엔터 버튼을 눌렀더니 검색어에 줄바꿈 기호가 포함되는 것 같더라구요! 요것도 추후 제거 되면 좋을 것 같아요
  1. 검색화면과 검색결과 화면 모두에서 검색 함수를 중복 호출하는 문제가 있었습니다.. 검색 결과화면의 경우 navigate할때 자동으로 검색화면전까지 singleTop을 유지하도록 되어있기 때문에 문제가 안보였으나, 검색화면에서 검색결과 화면으로 넘어가는 경우에 스택이 2개가 쌓이면서 생겼던 문제입니다. 이에 대해 중복 호출 제거로 해결하였습니다.
  2. 검색어 양끝 앞뒤로 개행문자를 비롯한 white space들을 모두 제거하는 메소드를 이용해 검색되도록 수정하였습니다.

@DongJun-H DongJun-H requested a review from yuni-ju June 2, 2025 12:10
@github-project-automation github-project-automation bot moved this from Todo to In Progress in DAYO 2.0 Jun 2, 2025
@DongJun-H DongJun-H merged commit fddef58 into develop Jun 3, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in DAYO 2.0 Jun 3, 2025
@DongJun-H DongJun-H deleted the bug/issue-658 branch June 3, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants