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

[CHORE] 마이페이지 러닝기록 / 제목 수정 시, 키보드 완료 버튼 누르면 바로 수정되도록 변경 #271

Merged
merged 1 commit into from
Nov 5, 2023

Conversation

leeeha
Copy link
Member

@leeeha leeeha commented Nov 5, 2023

📌 개요

✨ 작업 내용

  • 이슈 내용 확인 부탁드립니다!

✨ PR 포인트

굉장히 사소하지만 이런 걸로도 사용자 경험이 개선될 수 있다고 느꼈습니다!
코드 변경 사항 얼마되지 않으니 빠른 확인 부탁드립니다 :)

좋은 코드가 무엇인가를 고민하다가 개발 속도가 너무 더뎌지는 거 같아서, 이제는 좀 속도를 내보려고 합니다!!
데드라인 맞추는 것도 타 파트와의 협업에서 굉장히 중요하니까요!
그리고 이번 이슈처럼 자잘한 것들도 계속해서 개선해나가겠습니다! 화이팅!

@leeeha leeeha added 하은 🐰 하은 담당 CHORE 🛠 버전 코드, 패키지 구조, 함수 및 변수명 변경 등의 작은 작업 labels Nov 5, 2023
@leeeha leeeha requested a review from unam98 November 5, 2023 05:15
@leeeha leeeha self-assigned this Nov 5, 2023
Copy link
Contributor

@unam98 unam98 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다~!

Comment on lines 88 to +93
private fun addListener() {
initBackButtonClickListener()
initShowMoreButtonClickListener()
initEditFinishButtonClickListener()
initTitleEditorActionDoneListener()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

addListener에 포함되는 함수들을 아래에 순서대로 배치해주신 것 좋았습니다~!

@leeeha leeeha merged commit c8a8fdb into develop Nov 5, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CHORE 🛠 버전 코드, 패키지 구조, 함수 및 변수명 변경 등의 작은 작업 하은 🐰 하은 담당
Projects
2 participants