Skip to content

Conversation

@l5x5l
Copy link
Contributor

@l5x5l l5x5l commented Nov 30, 2024

Key Changes

  • 링크 아이템 표시 UI에 북마크 표시/메모/공유 여부 표시 아이콘 추가 및 체크박스 추가
  • 링크 아이템/케밥 버튼 클릭 이벤트 수정
  • 링크 상세 bottomSheet 디자인 추가

Resolves: #83

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

To Reviewers

  • 잘 안읽히거나 개선이 필요한 부분이 있다면 피드백 부탁해!

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 정해진 코딩 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트)

Etc.

링크 상세 bottomSheet에 대해, bottomSheet의 내용물 부분만 별도로 분리했는데(LinkDetailBottomSheetContent)
이는 bottomSheet를 한번 호출한 상태에서 다른 bottomSheet를 호출하는 로직을
아예 하나의 bottomSheet에서 그 안애 표시되는 내부 ui만 수정해서 사용할 수 있게 하기 위해서야!
(예- 링크 상세 bottomSheet에서 삭제 버튼을 클릭해 삭제 검사 bottomSheet로 표시하고자 하는 경우)

추가로 remind api쪽은 아직 링크 리스트를 전달받을 때 즐겨찾기정보 없이 들어와서 리마인드 화면에 대해서는 링크에 즐겨찾기 여부는 표시되지 않도록 해놨어

- 링크 상세 bottomSheet에서 bottomSheet를 제거한 부분을 별도의 LinkDetailBottomSheetContent로 분리
- 케밥 버튼 클릭시 링크 상세 bottomSheet 표시
- 링크 클릭시 해당 링크 url로 이동 (웹 브라우져)
@l5x5l l5x5l linked an issue Nov 30, 2024 that may be closed by this pull request
3 tasks
@l5x5l l5x5l requested a review from jiwon2724 November 30, 2024 09:56
Copy link
Member

@jiwon2724 jiwon2724 left a comment

Choose a reason for hiding this comment

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

세환 고생 많았어~~!
바루 머지해도 될 것 같아여!

@l5x5l l5x5l merged commit 464a6a3 into develop Dec 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] 링크 클릭 이벤트 및 링크 리스트, 링크 상세 bottomSheet UI 수정

3 participants