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 TargetListArea 퍼블리싱 #53

Merged
merged 8 commits into from
Jul 8, 2024
Merged

Conversation

wrryu09
Copy link
Member

@wrryu09 wrryu09 commented Jul 8, 2024

작업 내용 🧑‍💻

  • ArrangeBtn height 수정
  • TargetArea 추가

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 지민언니가 만든 공통컴포넌트 버튼 높이가 피그마에 있는 것이랑 달라서 피그마 버전으로 수정한 뒤 사용했습니다 !! 괜찮은지 확인 부탁드려요
  • 지민언니가 settings 페이지에 뿌려둔 버튼들 중 disabled 속성 전달하는 데에서 린트 오류가 나서 포맷팅이 조금 수정되었습니다 컨플릭나면 제 버전 무시하셔도 됩니다!
  • 내부 태스크 컴포넌트가 정확히 안 나왔고 제가 만든 버튼 피알도 아직 머지가 안 되어서 임시로 사용한 컴포넌트들이 많습니다
  • 지원언니가 만들어주기로 한 그라디언트 컴포넌트 나오면 붙이겠습니다
  • 스크롤바 정확한 디자인 나오면 커스텀 예정입니다

관련 이슈

close #52

스크린샷 (선택)

Jul-08-2024 12-37-01

@wrryu09 wrryu09 self-assigned this Jul 8, 2024
@wrryu09 wrryu09 linked an issue Jul 8, 2024 that may be closed by this pull request
1 task
Copy link
Member

@seong-hui seong-hui left a comment

Choose a reason for hiding this comment

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

모두 잘 되는 거 확인했습니당 수고하셨습니다!!

Copy link
Member

@jeeminyi jeeminyi left a comment

Choose a reason for hiding this comment

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

제가 만든 컴포넌트 린트 오류 수정하였습니다! 아마 제가 pr올리고 머지하면 될 것 같습니다.
중복되는 컴포넌트 명도 수정해두었습니다! 코드 잘 확인했습니다. 고생하셨습니다!!

@wrryu09 wrryu09 merged commit da99555 into develop Jul 8, 2024
1 of 2 checks passed
@wrryu09 wrryu09 deleted the feat/#52/targetlistarea-view branch July 8, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT TargetListArea 퍼블리싱
3 participants