-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FE][FEAT] #145 - 플로팅 버튼 구현 #150
Conversation
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment 남겨주신 부분은 전체적인 플로우를 확인해야 답변할 수 있을 것 같아서, 백엔드 구현 하고나서 보면 너무 늦어질 것 같아 저는 코드만 보고 수정하면 좋을 것 같은 부분 남겨뒀습니다!
코멘트 달아주신 내용은 재도님과 주원님께서 확인해보시고 결정하시는 게 더 빠르고 좋을 것 같습니다!
그리고 영상 캡처해주신 것만 확인하고 코드로는 확인하지 않아서 정확하지는 않은데, 디자인이 많이 반영되지 않은 것 같습니다! 토글 간의 간격도 조금 달라 보이고. 토글 간에 그림자도 넣어뒀었는데 그런 디테일이 반영되지 않은 것 같습니다! 확인 부탁드립니닷!
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
frontend/src/component/common/FloatingButton/FloatingButton.tsx
Outdated
Show resolved
Hide resolved
공통컴포넌트 이슈 관련해서는 크게 신경쓰지 않아도 될 것 같아요. FlatingButton에는 추가적인 요소만 들어갈 수 있도록 확장성만 고려해서, 도메인 종속적으로 만들어도 될 것 같아요. 근데 일케 적고보니 걍 공통컴포넌트 느낌이긴 하네요 ㅎㅎ... 공통컴포넌트가 완벽한 껍데기(보일러 플레이트)가 될 필요는 없다고 생각해요. 애니메이션이나 디자인들은 정해놓고 데이터와 이벤트만 다르게 받아도 여러 곳에서 재사용이 가능하겠죠! 그러면 그게 곧 공통컴포넌트라고 생각합니다. 만약 공통컴포넌트로 만든다고 하면, 제 방식으로는 의존성(데이터 등을)외부에서 전부 주입한다고 생각하고 작업하는 것 같아요. Dropdown을 기준으로 말하면, 드랍다운도 하나하나 다 Router를 걸어줘야하는데, 저는 이에 대해서는 데이터도 외부에서 넘겨주는 만큼 이벤트도 외부에서 하나만 넘겨주면 결과적으로 Dropdown은 배열 하나와 이벤트 하나 이렇게 받으면 되는 형태가 나올 것 같아서요. 다만 이게 좋은 방법인지는 고민좀 해보려고 합니다. |
…oostcampwm-2024/web28-DDara into feature/fe/#145-FloatingButton
b667d1a
📝 PR 개요
✅ 체크리스트 (Checklist)
🔄 관련 이슈 (Linked Issues)
📷 스크린샷 및 동영상 (선택 사항)
참고사항