-
Notifications
You must be signed in to change notification settings - Fork 0
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
Impl budget proposal page for reviewers #87
base: dev
Are you sure you want to change the base?
Conversation
이거 status 관리가 조금 헷갈리는데 리뷰어 페이지에서 승인 / 반려 / 수정 요청을 누르면 각각 어떤 상태로 바뀌어야 하는건가요?? |
현황 태그가 |
수정 완료~ |
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.
textArea를 아예 분리한건가요?
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.
네 기존의 textinput에다가 area 줘서 구현하는 방법은 placeholder가 안보여서 분리했어요
{/* {userPermission === 1 && <ViewerIncomeTable data={mockViewerIncomeData} />} */} | ||
{userPermission === 2 && ( | ||
<ReviewerIncomeTable initialData={mockIncomeData} /> | ||
)} | ||
{/* {userPermission === 3 && <ViewerIncomeTable data={mockViewerIncomeData} />} */} | ||
|
||
{/* {userPermission === 1 && <ViewerExpenditureTable data={mockViewerExpenditureData} />} */} |
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.
저게 지금은 유저 권한을 하드코딩으로 처리하다보니까 다른 부분을 주석처리하지 않으면 린트 에러가 나요
나중에 백엔드 연결하면 다 주석 풀면 됩니당
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.
테이블 테두리를 알맹이가 다 채우지 않는 문제가 보여서 (저한테만 그런 거라면 꼭 알려주세요!!) 코멘트 달아 놓았습니다!
id: "item", | ||
header: "항목", | ||
cell: info => info.getValue(), | ||
size: 275, |
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.
); | ||
return <DarkTag color={color}>{text}</DarkTag>; | ||
}, | ||
size: 180, |
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.
ReviewerIncomeTable에 단 코멘트랑 같은 맥락으로, 요기도 200 정도로 설정해주실 수 있을까요??
dropdown이 매번 4px이면 constant에 써놓고 변수로 가져다 쓰면 좋을 것 같아요! |
TableInner fit-content로 바꾸니까 테이블 크기를 내용물이 다 못채우던 문제가 해결된 것 같아용 관련해서 기존에 그냥 크게 잡아놨던 너비들 디자인이랑 맞춰볼게요
|
흠 그니까요.. 저거 왜저러냐..ㅠㅠㅠㅠ |
요약 *
It closes #86
스크린샷
이후 Task *