마이페이지 화면구성을 진행합니다. #20
Conversation
Conflicts: src/components/profileImage/ProfileImage.tsx
|
출생 연도만 선택하는 거면 dropdown으로 구현을 하는게 좋을 것 같습니다 |
|
마이페이지에 로그아웃 버튼을 하나 만들어야 할 것 같습니다! |
|
@sera2002 오 넵 세라님 드롭다운으로 출생년도 구현 및 로그아웃 버튼 추가하겠습니다! 좋은 의견 감사합니당 |
|
@sera2002 세라님 그러고보니 로그아웃 버튼은 상단 내비게이션 바 어디엔가 추가될 것 같아서 마이페이지에 없어도 될 것 같은데 어떻게 생각하시나용? |
@sera2002 드롭다운으로 구현 완료하였습니다!
|
음.. 이 서비스를 이용할 때 로그아웃/재로그인이 많이 일어나진 않을 것 같아서 굳이 상단 내비게이션 바에 로그아웃 버튼을 항상 노출을 시켜놓을 필요는 없을 것 같습니다. |
아하 넵넵! 마이페이지에 추가하겠습니다! |
src/feature/mypage/mypage.delete/components/MypageDeleteConfirmForm.tsx
Outdated
Show resolved
Hide resolved
| const age = dayjs().year() - Number(birthYear); | ||
| const ageText = `${age}세`; | ||
|
|
||
| const genderText = GENDER_LOOKUP_TABLE[gender] ?? "입력하지 않음"; |
There was a problem hiding this comment.
이거 테이블에 있는 U랑 "입력하지 않음"의 차이가 어떻게 될까요?
There was a problem hiding this comment.
@hoon5083 '입력하지 않음' 은 말그대로 입력하지 않은 상태, U 는 명시적으로 어떤 성별인지 드러내고 싶지 않은 상태를 의미한다고 생각하였습니다.
Conflicts: src/components/button/Button.tsx src/components/dropdown/Dropdown.tsx
src/feature/mypage/mypage.delete/components/MypageDeleteConfirmForm.tsx
Outdated
Show resolved
Hide resolved
src/feature/mypage/mypage.delete/components/MypageDeleteConfirmForm.tsx
Outdated
Show resolved
Hide resolved
|
|
||
| button { | ||
| margin-top: 32px; | ||
| // TODO: danger Button 머지되면 버튼에 danger prop 부여 |
There was a problem hiding this comment.
danger props 추가된거 머지된 상태여서 여기 수정하시면 될 것 같습니다!
There was a problem hiding this comment.
@sera2002 오 예리한 포인트군요 변경하였습니다!!
작성자: @jaychang99
Related to
체크 리스트
작업 내역
작업 중 공통 컴포넌트 변경 내역
<TextInput />컴포넌트가 이제valuesprops 의 변화도 감지하도록 하여 초기값을 가져오는 비동기 통신에 의한 데이터 변화를 감지할 수 있도록 하였습니다.<Button />컴포넌트가 외부의className도 받을 수 있도록 수정하였습니다.dayjs날짜 관리 라이브러리를 설치하여 보았습니다.비고
DatePicker가 아직 개발되지 않았는데 출생년도 선택이니까 그냥 드롭다운으로 해도 되지 않을까 라는 생각이 들었습니다. 그렇게 바꾸어 볼까요?