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/#76] 은행 컴포넌트(바텀시트) 구현 #78

Merged
merged 15 commits into from
Jul 10, 2024

Conversation

imddoy
Copy link
Contributor

@imddoy imddoy commented Jul 10, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 은행 input 컴포넌트 제작
  2. 은행 바텀시트 제작

📢 To Reviewers

  • props를 조금 넘기게 된 것 같아서 죄송합니다.... 그치만 최선이었다는 것을 알아주세용 좋은 방법 알려주시면 공부해서 바로바로 고치겠습니다!!!
  • 스토리 포기!

📸 스크린샷

image

입력 전 input창

image

은행 클릭시 바텀시트 오픈

image

입력 완료

🔗 참고 자료

Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

@github-actions github-actions bot added the ✨ FEAT 기능 구현 label Jul 10, 2024
Copy link

Copy link

Copy link
Contributor

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

새벽까지 해서 완성한 컴포넌트,,,,,, 고생 많았어요 ㅠㅠ 🫶 은행 바텀시트 function 때문에 어푸는 수정되면 하겠습니다!!

Copy link
Contributor

Choose a reason for hiding this comment

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

p3) tsx로 맞춰주셨다고 해서 혼란스럽지 않도록 이 친구는 지워주는 게 좋을 것 같습니다~~!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

삭제 완료했습니다~

`;

// BankBtn
export const BankBtnWrapper = styled.button`
Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 결국 성공한,,, 이게 진짜 의지의 한국인이다

const BankBottomSheet = ({ value, onBankClick, onOutClick }: BankBottomSheetProps) => {
return (
<>
<S.BankLayout>
Copy link
Contributor

Choose a reason for hiding this comment

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

p1) PR에 올려주신 사진을 보면 은행을 선택해주세요 라는 글자 위에 function이라는 작은 글자가 보이는데 혹시 의도한 게 아니라면 지워주셔야 할 것 같아요!! 혹시 보기 용이하게 하기 위한 익스텐션 같은 거라면,,,,, 민망해서 혀깨물겠습니다
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

헤헤 지우고.... 푸시를 까먹었어요...ㅎㅎ 완료했습니다!! 쵝오

`;

// InputBank
export const InputBank = styled.div<{ $hasChildren: boolean }>`
Copy link
Contributor

Choose a reason for hiding this comment

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

p3) 제 기분탓일지 모르겠지만,,, 은행을 선택해주세요 Input 글자가 살짝 위로 치우쳐진 것 같아요!!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다! 매의 눈!!!👁️👁️

return (
<S.InputAccountWrapper>
<S.InputAccountLabel>공연료 입금 계좌번호</S.InputAccountLabel>
<Spacing marginBottom="2" />
Copy link
Contributor

Choose a reason for hiding this comment

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

p5) 도영 오빠가 만들어준 공통 컴포넌트 야무띠게 사용하는 채현,,, 조아욥

Copy link
Contributor Author

Choose a reason for hiding this comment

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

도영리드 최고~ 칭찬해주는 신지는 그냥 내 사랑이나 먹어라.....

Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ;)
깔끔하네용!
리뷰만 확인하고 머지해도 될 것 같아요!

import styled from "styled-components";

// BankBottomSheet
export const BankLayout = styled.section`
Copy link
Member

Choose a reason for hiding this comment

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

image

p1) 여기서 사이즈가 안 맞아요! 이거 때문에 중앙 정렬이 되지 않는 것 같은데 확인 부탁드립니다!

`;

// InputBank
export const InputBank = styled.div<{ $hasChildren: boolean }>`
Copy link
Member

Choose a reason for hiding this comment

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

p4) 이 컴포넌트 누르면 바텀시트가 열리니까 여기에 cursor: pointer; 추가하면 좋을 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

바녕완

Copy link
Member

Choose a reason for hiding this comment

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

p5) 굿 ~

@pepperdad
Copy link
Member

TODO: 개발자 도구 키면 발생하는 오류 -> 배포 환경에서 테스트

@pepperdad pepperdad changed the title [ Feat/#76 ] 은행 컴포넌트(바텀시트) 구현 [Feat/#76] 은행 컴포넌트(바텀시트) 구현 Jul 10, 2024
Copy link

@imddoy imddoy merged commit 26f7aff into develop Jul 10, 2024
2 checks passed
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ FEAT 기능 구현
Projects
Status: 🎉Done
Development

Successfully merging this pull request may close these issues.

[ Feat ] Bank 컴포넌트 제작
3 participants