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

Impl budget proposal page for reviewers #87

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

malloc3141
Copy link
Contributor

요약 *

It closes #86

스크린샷

이후 Task *

  • 없음

@malloc3141 malloc3141 added enhancement New feature or request frontend labels Jan 24, 2025
@malloc3141 malloc3141 self-assigned this Jan 24, 2025
@malloc3141 malloc3141 marked this pull request as ready for review January 25, 2025 01:49
@malloc3141
Copy link
Contributor Author

이거 status 관리가 조금 헷갈리는데 리뷰어 페이지에서 승인 / 반려 / 수정 요청을 누르면 각각 어떤 상태로 바뀌어야 하는건가요??

@ChaeyeonAhn
Copy link
Contributor

현황 태그가
검토 승인 / 수정요청 / 검토반려
이렇게 되어야행ㅅ!

@malloc3141
Copy link
Contributor Author

수정 완료~

Copy link
Contributor

Choose a reason for hiding this comment

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

textArea를 아예 분리한건가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 기존의 textinput에다가 area 줘서 구현하는 방법은 placeholder가 안보여서 분리했어요

Comment on lines +224 to +230
{/* {userPermission === 1 && <ViewerIncomeTable data={mockViewerIncomeData} />} */}
{userPermission === 2 && (
<ReviewerIncomeTable initialData={mockIncomeData} />
)}
{/* {userPermission === 3 && <ViewerIncomeTable data={mockViewerIncomeData} />} */}

{/* {userPermission === 1 && <ViewerExpenditureTable data={mockViewerExpenditureData} />} */}
Copy link
Contributor

Choose a reason for hiding this comment

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

주석 처리한 부분은 뭔가요?

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
Contributor

@ChaeyeonAhn ChaeyeonAhn left a 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,
Copy link
Contributor

Choose a reason for hiding this comment

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

제가 열어서 확인해 봤을 때는 테이블이 지금 테두리를 알맹이가 다 안 채워주는 것 같더라구요
혹시 요기를 size: 400 이렇게 충분히 큰 숫자로 채워주실 수 있으실까요?

스크린샷 2025-02-03 오후 10 19 37

);
return <DarkTag color={color}>{text}</DarkTag>;
},
size: 180,
Copy link
Contributor

Choose a reason for hiding this comment

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

ReviewerIncomeTable에 단 코멘트랑 같은 맥락으로, 요기도 200 정도로 설정해주실 수 있을까요??

@ChaeyeonAhn
Copy link
Contributor

ChaeyeonAhn commented Feb 3, 2025

수고 많으셨습니다 ㅠㅠ

제가 열어서 확인해 보니까 조회 가이드 만들 때의 제 실수 때문에 이상해진 부분도 있고 디자인 구현을 위해 부탁드리고 싶은 게 있어서,
아래의 내용들 반영해주시면 감사하겠습니다!

  1. SemesterSelectCard.tsx의 CardContent 안에 justify-content: center;
    코드를 저 대신 추가해 주시면 감사하겠습니다아..
    요래 왼쪽으로 치우쳐 있어서 해결이 필요할 것 같아요
스크린샷 2025-02-03 오후 10 47 18
  1. ViewResult.tsx의 DropdownList 안의 transform: translateX(-20px) translateY(+4px); 요렇게 y가 4px 내려가도록 수정 부탁드립니다! (우리 모든 드롭다운은 4px 내려온다고 생각하시면 됩니다 ㅎㅎ)

  2. Table.tsx의 TableInner의 width: fit-content;로 수정 부탁드립니다!
    아래 첫번째 영상처럼 되지 않게 하고, 두번째 처럼 되면 좋을 것 같아요!
    요거 써보시면 아마 요렇게 될텐데 아니라면 꼬옥 알려주세요!

2025-02-03.10.41.12.mov
2025-02-03.10.37.22.mov

@wjeongchoi
Copy link
Contributor

dropdown이 매번 4px이면 constant에 써놓고 변수로 가져다 쓰면 좋을 것 같아요!

@malloc3141
Copy link
Contributor Author

malloc3141 commented Feb 4, 2025

TableInner fit-content로 바꾸니까 테이블 크기를 내용물이 다 못채우던 문제가 해결된 것 같아용

관련해서 기존에 그냥 크게 잡아놨던 너비들 디자인이랑 맞춰볼게요

  • 바꾸니까 다시 저 문제가 생기네요.. 근데 기존에 맞춰놓은 너비로는 내용이 너무 늘어져서 보이는 문제가 있긴 해요. 어떻게 해결하면 좋을까요..??

@ChaeyeonAhn
Copy link
Contributor

TableInner fit-content로 바꾸니까 테이블 크기를 내용물이 다 못채우던 문제가 해결된 것 같아용

관련해서 기존에 그냥 크게 잡아놨던 너비들 디자인이랑 맞춰볼게요

  • 바꾸니까 다시 저 문제가 생기네요.. 근데 기존에 맞춰놓은 너비로는 내용이 너무 늘어져서 보이는 문제가 있긴 해요. 어떻게 해결하면 좋을까요..??

흠 그니까요.. 저거 왜저러냐..ㅠㅠㅠㅠ
애초에 responsive가 너무 패딩이 작나

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impl budget proposal page for reviewers
3 participants