-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FEAT/#220] 2차 스프린트 뷰 / 프로필, 확장함수, CustomEtv 전체 삭제 기능 구현 #222
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
후 이제 괴롭힐게 없음 ..
@@ -79,6 +79,7 @@ object ThirdPartyDependencies { | |||
const val amplitude = "com.amplitude:android-sdk:${Versions.amplitudeVersion}" | |||
|
|||
const val flexbox = "com.google.android.flexbox:flexbox:${Versions.flexboxVersion}" | |||
const val circleImageView = "de.hdodenhof:circleimageview:${Versions.circleImageViewVersion}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와 보니깐 아워투두 친구목록에도 테두리 있었네 머지하면 쓰겟삼 최고~~
btnDeleteText.setOnClickListener { | ||
etEmojiCounterEtContent.text = null | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
물론 성능에 크게 영향은 없다만, 싱글클릭으로 사용하는거는 어떨까요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넹~ 수정했습니당!
근데 혹시 이유를 알 수 있을까요??
저는 이 버튼이 클릭 하는 순간 사라지기 때문에 굳이 싱글클릭을 안해도 될 것 같다 생각해서 그냥 클릭 리스너로 했습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사실 상관없습니다! ㅋㅋ
const val NICKNAME = "NICKNAME" | ||
const val INFO = "INFO" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
createIntent를 활용하면 키값을 private으로 돌려도 작동함당 !
그것이... 캡슐화...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
private 했습니다... 그것이..캡슐화니까...
android:layout_width="0dp" | ||
android:layout_height="0dp" | ||
app:layout_constraintEnd_toEndOf="parent" | ||
app:layout_constraintHeight_percent="0.23" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
깔꼼하고 좋습니다!!! 굿굿~
implementation(flexbox) | ||
implementation(circleImageView) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋아요!!
private fun observeNameTextChanged() { | ||
binding.etProfileEditNickname.editText.doAfterTextChanged { name -> | ||
viewModel.checkIsNameChanged(name.toString()) | ||
} | ||
} | ||
|
||
private fun observeInfoTextChanged() { | ||
binding.etProfileEditInfo.editText.doAfterTextChanged { info -> | ||
viewModel.checkIsInfoChanged(info.toString()) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오,,,doAfterTextChanged
를 사용해서 검사할 수도 있군요?!
fun setDefaultValues(name: String, info: String) { | ||
nowName = name | ||
nowInfo = info | ||
|
||
defaultName = name | ||
defaultInfo = info | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아하 요렇게 디폴트 값으로 이름이랑 정보 넣어두고 검사하는 로직이군여!!
private fun setEtNameArguments() { | ||
with(binding.etProfileEditNickname) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 with()
를 이렇게 쓰니 코드가 간결하네욤
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
근데 진심 이거 쩌는 것 같아요...... 확 간결해지는 마법
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
역시!!! 수고하셨습니다👍👍👍👍최고최고
binding.btnTendencySplashStart.setOnSingleClickListener { | ||
navigateToScreen<TendencyTestActivity>(listOf(Intent.FLAG_ACTIVITY_CLEAR_TOP)) | ||
navigateToScreenClear<TendencyTestActivity>() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋습니다!! 만들어주신 확장함수 유용하게 잘 쓰고 있어요!!
private fun setEtNameArguments() { | ||
with(binding.etProfileEditNickname) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
근데 진심 이거 쩌는 것 같아요...... 확 간결해지는 마법
⛳️ Work Description
📸 Screenshot
📢 To Reviewers