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

[mod] 상세 피드 댓글 / 1차 릴리즈 QA 피드백 반영 #158

Merged
merged 11 commits into from
Aug 27, 2023

Conversation

leeeha
Copy link
Member

@leeeha leeeha commented Aug 26, 2023

📝 Work Description

  • 작업 내용은 이슈 확인해주세요!
  • 마이페이지 사용자 이용약관 링크 수정했습니다.
  • 회원탈퇴 버튼 수정하는 거 매우 간단한 작업이라 따로 브랜치 파지 않고 작업했습니다.
  • 위니 업로드 마진, 패딩 값 수정 사항도 반영했습니다!

📣 To Reviewers

  • 오늘 저녁이나 자정까지 리뷰 부탁드려요!

@leeeha leeeha added mod 〰 코드 및 내부 파일 수정 ui 🎨 UI 관련 작업 하은 🐰 labels Aug 26, 2023
@leeeha leeeha added this to the 기능 구현 ✨ milestone Aug 26, 2023
@leeeha leeeha self-assigned this Aug 26, 2023
Copy link
Contributor

@Sangwook123 Sangwook123 left a comment

Choose a reason for hiding this comment

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

👍👍

Copy link
Contributor

@sxunea sxunea 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 79 to 86
private fun initEditTextFocusChangeListener() {
binding.etComment.setOnFocusChangeListener { view, hasFocus ->
if (hasFocus) {
val commentListSize = commentAdapter.currentList.size
binding.rvDetail.smoothScrollToPosition(commentListSize + 1)
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 어떻게 할지 궁금했었는데 setOnFocusChangeListener 방법 너무 좋네요

@@ -70,6 +70,7 @@ class DetailActivity : BindingActivity<ActivityDetailBinding>(R.layout.activity_

initEditTextFocusChangeListener()
updateStatusBarColor()
binding.tvCommentCreate.bringToFront()
Copy link
Contributor

Choose a reason for hiding this comment

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

이 속성도 배우고갑니다 ❤️

@leeeha leeeha merged commit 5c3356f into develop Aug 27, 2023
1 check passed
@leeeha leeeha deleted the feature/mod-detail-comment branch August 29, 2023 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mod 〰 코드 및 내부 파일 수정 ui 🎨 UI 관련 작업 하은 🐰
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[ui] 회원탈퇴 / 버튼 위치 변경 [mod] 상세 피드 댓글 / 1차 릴리즈 QA 피드백 반영
3 participants