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

feat: card, popup componenets #10

Merged
merged 12 commits into from
Jan 17, 2024
Merged

Conversation

isseul
Copy link
Collaborator

@isseul isseul commented Jan 15, 2024

우선 리뷰작성팝업, 대여신청팝업을 위한 WriteDialog 컴포넌트를 만들었습니다.

image

메시지랑 placeholder 설정 가능합니다.

@isseul
Copy link
Collaborator Author

isseul commented Jan 15, 2024

옷장 생성/수정 팝업 컴포넌트 만들었습니다.
옷장 아이콘이 없어서 적절한 이미지로 수정해야 할 것 같습니다.

image

@readygetset
Copy link
Collaborator

dialog borderRadius 변경 안 되나요?

handleCancel: React.MouseEventHandler<HTMLButtonElement>;
isOpen: boolean;
}
const ConfirmDialog: React.FC<ClosetDialogProps> = ({ handleSubmit, handleCancel, isOpen }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이름 수정해주세요

onChange={handleChange}
sx={{ backgroundColor: 'white', borderRadius: 1, margin: 3 }}
/>
<CancelSubmitBtns handleSubmit={handleSubmit} handleCancel={handleCancel} submitBtnText="등록" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

텍스트도 props로 받는 게 좋을 거 같습니다 수정/등록 두 개 다 써서요

handleCancel: React.MouseEventHandler<HTMLButtonElement>;
isOpen: boolean;
}
const ConfirmDialog: React.FC<WriteDialogProps> = ({ message, placeholder, handleSubmit, handleCancel, isOpen }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

마찬가지로 이름 변경해주세요

name="text"
type="text"
multiline
rows={15}
Copy link
Collaborator

Choose a reason for hiding this comment

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

크기 좀 줄여도 될 거 같아요 리뷰나 신청멘트 쓰는 거라 그렇게 길지 않을 거 같아서요

@isseul
Copy link
Collaborator Author

isseul commented Jan 16, 2024

ApplyCard입니다.
image

@isseul
Copy link
Collaborator Author

isseul commented Jan 16, 2024

피드백 반영해서 수정했습니다.

image

image

@readygetset
Copy link
Collaborator

신청 카드 이미지도 borderRadius 조정해주시면 좋을 거 같아요 그리고 뭔가 아웃라인이 있는 게 깔끔할 거 같습니당 그리고 width 쬐끔만 줄여주시면 더 이쁠 거 같아요 ~

@isseul
Copy link
Collaborator Author

isseul commented Jan 16, 2024

대여 내역 카드입니다.

  1. loanee인 경우
    1-1. 리뷰 있을 때
    image

1-2. 리뷰 없을 때
image

  1. lender인 경우
    2-1. 리뷰 있을 때
    image

2-2. 리뷰 없을 때
image

  1. 리뷰 작성하기 클릭 시 placeholder보이도록
    image

  2. 리뷰 수정 클릭 시 기존에 작성된 리뷰가 보이도록
    image

로그인 정보가 제대로 안 넘어가는 것 같아서 그것만 수정하면 될 것 같습니다.

@readygetset
Copy link
Collaborator

리뷰 작성하기 버튼이랑 리뷰 글씨랑 줄 안 맞는 거 같은데 조정해주시면 좋을 거 같습니다! 그거 빼면 다 좋네용

@isseul
Copy link
Collaborator Author

isseul commented Jan 17, 2024

image

리뷰 작성하기 버튼 줄 맞췄습니다.

Copy link
Collaborator

@readygetset readygetset left a comment

Choose a reason for hiding this comment

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

LGTM!

@isseul
Copy link
Collaborator Author

isseul commented Jan 17, 2024

api 잘 작동하는 것 확인했습니다.
image

review?: string;
}

const lendInfo: LendInfo = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

테스트한 건 빼고 머지해주세용

enqueueSnackbar(REVIEW_MESSAGE.REVIEW_NOT_FOUND, { variant: 'error' });
}
}
return null;
Copy link
Collaborator

Choose a reason for hiding this comment

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

없어도 됩니다

@isseul isseul requested a review from readygetset January 17, 2024 13:20
@readygetset
Copy link
Collaborator

그리고 혹시 옷장 아이콘 import CheckroomIcon from '@mui/icons-material/Checkroom'; 이걸로 변경 가능할까요? 이게 더 나을 거 같아서용

@isseul isseul merged commit 1465364 into develop Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants