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

[feat/#138] 카드 뒷면 관심사 삭제/추가 디테일 수정 #139

Merged
merged 5 commits into from
Feb 12, 2024

Conversation

unam98
Copy link
Collaborator

@unam98 unam98 commented Feb 12, 2024

📌 개요

✨ 작업 내용

  • 선택 가능한 관심사 최대 3개로 제한
  • 1개 이하 삭제 제한 토스트 메세지 수정
  • 삭제, 추가 시 itemDecoration 충돌 나면서 간격 이상해지는 거 해결
  • data 추가 시 기존 data가 안 날아가고 쌓일 수 있게 하기

✨ PR 포인트

  • 커스텀뷰 사용법이 친절하지 못하고 깔끔하지 못한 것 같아 (ex. backCardView.interestAdapter.onAddItemClick) 추후 싹 리팩토링을 해보겠습니다.
  • 관심사 추가/삭제 서버 api 연결은 직접 하셔야 됩니다.

📸 스크린샷/동영상

KakaoTalk_Video_2024-02-12-14-26-41.mp4
isModifyDetail 활성화 및 data 넣는 법 관심사 '추가' 터치 이벤트 처리

관심사 item이 삭제되거나 추가되면 어댑터의 currentList에 변화가 생길 텐데 이를 옵저빙할 수 있는 LivaData를 만들었습니다.
ex) backCardView.currentList.observe(this){} 이런식으로 Activity나 Fragment에서 옵저빙하다가 갱신된 관심사 data list를 콜백으로 받아오면 유저 정보 갱신 api에 넣어서 서버에 반영해주면 됩니다

스크린샷 2024-02-12 오후 10 07 57

스크린샷 2024-02-12 오후 10 07 43

@unam98 unam98 added 우남 FEAT 새로운 기능 구현 FIX 버그 및 오류 해결 labels Feb 12, 2024
@unam98 unam98 self-assigned this Feb 12, 2024
Copy link
Member

@blueme0 blueme0 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~

그리고 저는 카드 전면 뷰들은 다 클릭 리스너가 잘 붙여지는데 후면에 속한 뷰들은 클릭 리스너 말고 뒤집기가 작동해요 . ... 적용할 때 제가 살펴봐야 할 부분이 있을까여?

@unam98
Copy link
Collaborator Author

unam98 commented Feb 12, 2024

수고하셨습니다~~

그리고 저는 카드 전면 뷰들은 다 클릭 리스너가 잘 붙여지는데 후면에 속한 뷰들은 클릭 리스너 말고 뒤집기가 작동해요 . ... 적용할 때 제가 살펴봐야 할 부분이 있을까여?

터치 focus가 겹치는 것 때문인데 방법을 찾아보겠습니다

@unam98 unam98 merged commit bda5f80 into main Feb 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEAT 새로운 기능 구현 FIX 버그 및 오류 해결 우남
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 카드 뒷면 관심사 삭제 디테일 보충
2 participants