#46 [SC-1] 스크랩-리스트

[HyunjeLee]

📍 PR 타입 (하나 이상 선택)

• [x] 기능 추가
• [ ] 버그 수정
• [ ] 코드 리팩토링 or 수정
• [ ] 의존성, 환경 변수, 빌드 관련 코드 업데이트
• [ ] 기타 사소한 수정

❗️ 관련 이슈 링크

• 46

📌 개요

• 스크랩 리스트뷰 구현

🔁 변경 사항

• 제목과 링크 사이 간격은 가변이 아니라 고정으로 구현했습니다.
• 검색 기능은 추후 구현

📸 구현 화면 스크린샷

👀 기타 더 이야기해볼 점

• 스크랩 클릭 시 세부로 넘어가게끔 구현하는 게 어떨까요?
• 이미지는 centerCrop으로 중앙 기준으로 이미지박스에 맞춰 자르기입니다.
• 이미지 크기가 가변입니다.
• 그림자는 일단 넣었습니다.

✅ 체크 리스트

• [x] PR 템플릿에 맞추어 작성했어요.
• [x] 변경 내용에 대한 테스트를 진행했어요.
• [x] 프로그램이 정상적으로 동작해요.
• [x] PR에 적절한 라벨을 선택했어요.
• [x] 불필요한 코드는 삭제했어요.
• [x] 담당자를 할당했어요.
• [x] 리뷰어를 할당했어요.
• [ ] ✨일요일 20:00 까지 모든 리뷰에 대해
  1. 해결하거나
  2. 따로 이슈로 빼두고 노션에 공지했어요. (이슈 해결기간 제외)
• [x] android:exported 속성 확인

[choign3123]

검색 기능은 추후 구현

1. 검색 기능은 API 통신입니다. (페이징 처리 때문)
2. 검색 버튼 클릭시 '[SCR-1] 스크랩-리스트-검색결과'로 화면 전환됩니다. 자세한 내용은 짝코딩때 설명드릴게요

[choign3123]

스크랩 클릭 시 세부로 넘어가게끔 구현하는 게 어떨까요?

피그마에 그렇게 화면이동 설정해놨습니다. 노션에도 적어놨는데 혹시 못보셨을까용?

[HyunjeLee]

검색 기능은 추후 구현

1. 검색 기능은 API 통신입니다. (페이징 처리 때문)
2. 검색 버튼 클릭시 '[SCR-1] 스크랩-리스트-검색결과'로 화면 전환됩니다. 자세한 내용은 짝코딩때 설명드릴게요

화면이 따로 있어서 지금 화면에서는 구현 안하는 거 맞죠?

[HyunjeLee]

스크랩 클릭 시 세부로 넘어가게끔 구현하는 게 어떨까요?

피그마에 그렇게 화면이동 설정해놨습니다. 노션에도 적어놨는데 혹시 못보셨을까용?

지금은 스크랩의 제목이나 '>' 클릭 시 화면이동인데 그게 아니라 스크랩 아이템 전체 아무 곳이나 클릭해도 화면 이동하게끔 구현하는 게 어떤 가라는 질문입니다. 이미지는 그대로 링크 이동하게끔 구현하구요

[choign3123]

수정해야되는 부분이 글로 설명하는것보단 짝코딩할 때 설명드리는게 더 나을 것 같아서, 그때 나머지 부분 설명드리겠습니당~! 좋은 밤 보내세요 🌙

1. 이미지 테두리가제대로 보이지 않음.
2. '>' 버튼 없애는게 어떨지
3. 제목과 링크사이 간격