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

#19,47 / 소원권 ui & 로직 #62

Merged
merged 9 commits into from
Jul 19, 2023
Merged

#19,47 / 소원권 ui & 로직 #62

merged 9 commits into from
Jul 19, 2023

Conversation

sub101
Copy link
Member

@sub101 sub101 commented Jul 19, 2023

📌 관련 이슈

#19, #49

📷 screenshot

2023-07-19.2.37.12.mov

📝 Work Desciption

  • 소원권, 소원권 만들기, 나의 소원권, 상대 소원권 확인, 소원권 사용 ui 만들었습니다
  • edittext 제한 걸었습니다
  • 리사이클러뷰 아이템 선택시 화면 전환 구현 하였습니다.
  • 버튼 처리 로직을 구현 하였습니다.

📚 Reference 혹은 궁금한 사항들

�- edittext 엔터 처리 부분 추후에 다시 해야할 것 같습니다.

  • 아이템 포지션마다 다르게 페이지 전환 처리하는 것은 api 붙이면서 작업할 예정입니다.

@sub101 sub101 self-assigned this Jul 19, 2023
@sub101 sub101 added feat 구현 및 개선 UI UI 작업 수빈 🦄 유니의 외모 담당 labels Jul 19, 2023
@sub101 sub101 added this to the Phase2 milestone Jul 19, 2023
@sub101 sub101 changed the title #19,49 / 소원권 ui & 로직 #19,47 / 소원권 ui & 로직 Jul 19, 2023
Copy link
Contributor

@taeheeL taeheeL left a comment

Choose a reason for hiding this comment

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

고생했어요~

import sopt.uni.databinding.ActivityWishBinding
import sopt.uni.util.binding.BindingActivity

class WishActivity : BindingActivity<ActivityWishBinding>(R.layout.activity_wish) {

private lateinit var multiviewAdapter: WishMultiviewAdapter
Copy link
Contributor

Choose a reason for hiding this comment

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

어뎁터 객체를 생성했으면 꼭 onDestroy에 null 처리를 해줘야 메모리 릭이 나지 않습니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

넹넹 반영하겠습니다!

Comment on lines 80 to 81
val intent = Intent(this, WishFcActivity::class.java)
startActivity(intent)
Copy link
Contributor

Choose a reason for hiding this comment

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

우리에겐 확장함수가 있으니 사용하시면 편하실거에유

(holder as MultiViewWishHolder1).bind(datas[position])
when (holder) {
is MultiViewWishHolder1 -> {
holder.bind(datas[position])
holder.setIsRecyclable(false)
Copy link
Contributor

Choose a reason for hiding this comment

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

이 메소드는 뭐죵

Copy link
Member Author

Choose a reason for hiding this comment

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

RecyclerView의 각 아이템에 해당하는 데이터를 ViewHolder에 바인딩하고, 클릭 이벤트 처리하는 메소드인데 서버연결하면서 수정할것 같아용

Comment on lines 22 to 34
// with(binding) {
// dialogTitle.setText(getString(R.string.use_wish_dialog_title))
// dialogBody.setText(getString(R.string.use_wish_dialog_description))
// btnRight.setText(getString(R.string.dialog_ok_text))
// btnLeft.setOnSingleClickListener {
// dismiss()
// }
// btnRight.setOnSingleClickListener {
// val fragment = WishUsedFragment()
// parentFragmentManager.beginTransaction()
// .replace(R.id.fcv_wish_fc, fragment).commit()
// }
// }
Copy link
Contributor

Choose a reason for hiding this comment

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

주석 제거 plz

import sopt.uni.R
import sopt.uni.databinding.FragmentWishUseMyBinding

class WishUseMyFragment : Fragment() {
Copy link
Contributor

Choose a reason for hiding this comment

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

BindingFragment 상속받으면 좋아유

Copy link
Member Author

Choose a reason for hiding this comment

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

넹넹 추후에 전부 수정할게용!

@sub101 sub101 merged commit da99afa into develop Jul 19, 2023
1 check passed
@taeheeL taeheeL deleted the feature/#19-wishUi branch July 19, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 구현 및 개선 UI UI 작업 수빈 🦄 유니의 외모 담당
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants