-
Notifications
You must be signed in to change notification settings - Fork 7
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] 생성페이지 달력, 시간 입력 리팩토링(#603) #614
Conversation
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.
드디어 방 생서 페이지가 리팩터링 되었군요!!!
이제 디자인을 고민할 시간,,ㅎ 같이 고민해봐요
작업하느라 수고하셨어요~ 👍
@@ -6,7 +6,7 @@ export const TimeDropdownContainer = styled.section` | |||
width: 100px; | |||
`; | |||
|
|||
export const TimeDropdownToggle = styled.input<{ $error: boolean }>` |
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.
제출 시간보다 이전 시간을 선택하면 error 상태로 판단할 수 있을 것 같아요!
아직 방 생성과 수정이 오픈되지 않는 기능이라 추후에 반영하면 좋을 것 같아요
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.
문구랑 함께 띄워주면 좋을 것 같네요 ! 퍼블릭 기능으로 전환할 때 참고하겠습니다~~
const [recruitmentDate, setRecruitmentDate] = useState(new Date()); | ||
const [reviewDate, setReviewDate] = useState(new Date()); | ||
const [recruitmentTime, setRecruitmentTime] = useState(new Date()); | ||
const [reviewTime, setReviewTime] = useState(new Date()); |
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.
오 복잡했던 이 4가지 상태가 사라지니 너무 좋네요 ㅎㅎ
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.
ㅋㅋㅋㅋㅋㅋㅋ사라져서 다행이에요........
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.
✅ 방 생성 로직이 정리됐네요 ㅎㅎ
export const formatCombinedDateTime = (date: Date): string => { | ||
return `${formatDate(date)} ${formatTime(date)}`; |
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.
굿굿 드디어 개선이 됐네요 👍
@@ -218,10 +188,12 @@ const RoomCreatePage = () => { | |||
<S.ContentInput> | |||
<Input | |||
type="number" | |||
min="1" |
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.
이것도 생각보다 불편했는데 추가됐네요 굿굿
📓 스토리북 링크
바로가기
📌 관련 이슈
✨ PR 세부 내용
DateTimePicker
라는 컴포넌트를 만들어서 달력과 시간을 한 번에 입력 받도록 하였습니다.CreateRoomPage
전체적으로 리팩토링 하였습니다. (handleInputChange 함수 간소화, isFormValid 변수 추가)