-
Notifications
You must be signed in to change notification settings - Fork 3
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] MyPage 탭 UI 구현 #22
Conversation
string 추가
Feature/#11 my view my page
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.
넘 잘하시네용,,, 최고입니다 👍❤️🔥
class ChooseProjectBottomFragment : | ||
BottomSheetDialogFragment<FragmentBottomChooseProjectBinding>(R.layout.fragment_bottom_choose_project) { | ||
|
||
private val dummyItemList = mutableListOf<ResponseMyPageProjectDto>( |
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.
p3) 되도록이면 데이터를 갖는 리스트는 뷰모델에 넣어주세용 !
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.
앗 더미데이터라고 걍 아무데나 넣었네용... 머쓱머쓱
chooseProjectBottomFragment.show(requireActivity().supportFragmentManager, "show") | ||
} | ||
}) | ||
val concatAdapter = ConcatAdapter(myRetroTopAdapter, myRetroAdapter) |
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.
어댑터명은 해당 뷰의 어떤 부분인지 확실하게 명시해주는게 좋을 것 같아요 ! ex) titleAdapter, contentAdapter
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.
추천 감사합니닷
}) | ||
val concatAdapter = ConcatAdapter(myRetroTopAdapter, myRetroAdapter) | ||
|
||
with(binding) { |
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(binding) { | |
with(binding.rcvMyRetroMain) { |
holder.onBind(nickName) | ||
} | ||
|
||
class TopViewHolder(private val binding: ItemMypageNicknameBinding) : |
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.
p2) viewHolder명은 어댑터명 그대로 가는게 좋습니당
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.
헉 왜 다르게 했지..? 짚어주셔서 감사해욧
mypage 패키지는 home 밖으로 빼주세용 피그마 파란 부분 기준으로 패키지 나누는걸로 ! |
✍️ 작업한 내용
✔️ PR point
📸 스크린샷
KakaoTalk_20230715_015818661.mp4
🍀 관련 이슈