Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kuring-122 compose 키보드 이슈 업데이트 및 모듈간 sdk 의존성 통합 #89

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

yeon-kyu
Copy link
Member

@yeon-kyu yeon-kyu commented Jan 28, 2024

https://kuring.atlassian.net/browse/KURING-122

배경

  • compose에서는 textField 에 요상한 버그가 있어서 수정합니다 (View의 EditText가 벌써부터 그리워지네요😫)
    • 사용자가 텍스트 중간에 한글을 입력하려 하면 커서가 맨뒤로 가지는 이슈(영어는 이슈 재현안됨)
    • 관련 Issue Tracker 링크
  • 피드백 화면 뿐만 아니라 검색화면에서도 지장이 있어, 사용자에게 안좋은 경험을 줄 수 있어 라이브러리 업데이트를 진행했습니다.

변경사항

  • 각 모듈별 compileSdk, targetSdk, minSdk 를 Gradle Version Catalog에 의존하도록 합니다
  • compose-bom 을 2024.01.00 으로 업데이트합니다. 이 과정에서 아래의 추가 업데이트/대응이 필요
    • targetSdk를 34로 업데이트
    • Subscriptions.kt 에서 HorizontalPager 사용처 업데이트 대응 84cd311

Screen Shot

as-is

Screen_recording_20240128_170722.mp4

to-be

Screen_recording_20240128_170439.mp4

@auto-assign auto-assign bot requested a review from mwy3055 January 28, 2024 08:19
Copy link
Member

@mwy3055 mwy3055 left a comment

Choose a reason for hiding this comment

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

별 에러가 다 있네요;; 변경사항 확인했습니다. 라이브러리 업데이트는 언젠가 하려 했었는데, 마침 잘 됐네요.

간헐적으로 테스트가 실패하는 문제는 따로 PR 올릴게요

@yeon-kyu yeon-kyu merged commit b72c4dc into 2.0/base Jan 30, 2024
2 checks passed
@mwy3055 mwy3055 deleted the 2.0/kuring-122_yk branch January 31, 2024 05:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants