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

SKRF-447 행사정보 작성페이지 2차 QA #206

Merged
merged 10 commits into from
Dec 1, 2023

Conversation

SongInjae
Copy link
Collaborator

📝요구사항과 구현내용

  • 작업 내용 1: 공연 생성시 maxTicketCount 없을 시, 1로 들어가도록 수정
  • 작업 내용 2: 수정 불가능한 값들 readOnly 처리
  • 작업 내용 3: 행사 생성시 내용 max 200자->1000자 수정
  • 작업 내용 4: 행사정보 작성페이지 반응형 구현
  • 작업 내용 5: 행사정보 작성페이지 정원과 인당예매수 대소비교 예외처리

구현 스크린샷

스크린샷 2023-11-30 오후 10 45 25 스크린샷 2023-11-30 오후 10 46 29

✨pr포인트 & 궁금한 점

  • 반응형 웹페이지하면서 코드가 살짝 더러워졌는데 조금이라도 더 깔끔하게 바꿀 수 있을까요?

@SongInjae SongInjae added the enhancement New feature or request label Nov 30, 2023
@SongInjae SongInjae self-assigned this Nov 30, 2023
hyesung99
hyesung99 previously approved these changes Dec 1, 2023
Comment on lines 5 to 44
const EventFormContainer = styled.form`
display: flex;
position: relative;
flex-wrap: wrap;
width: 100%;
height: 100%;
padding: 3rem 0;

& > div:first-of-type {
padding-right: 1.5rem;
border-right: 1px solid ${Theme.color.tSeparator};
@media (min-width: 1023px) {
padding-right: 1.5rem;
border-right: 1px solid ${Theme.color.tSeparator};
}
}

& > div:nth-last-of-type(2) {
padding-left: 1.5rem;
@media (min-width: 1023px) {
padding-left: 1.5rem;
}
}
& > div:first-child > div:nth-last-of-type(2) {
flex-grow: 1;
}
`;
const ContentArea = styled.div`
width: 50%;
min-width: 50%;
min-height: 100%;
display: flex;
flex-direction: column;
gap: 1rem;

& > div:last-of-type {
@media (max-width: 1023px) {
padding-bottom: 1.5rem;
margin-bottom: 2rem;
}
}
`;
const TwoInputContainer = styled.div`
display: flex;
Copy link
Collaborator

Choose a reason for hiding this comment

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

const EventFormContainer = styled.form`
  display: flex;
  position: relative;
  flex-wrap: wrap;
  width: 100%;
  height: 100%;
  padding: 3rem 0;
  gap: 1rem;
`;
const ContentArea = styled.div`
  flex-grow: 1;
  display: flex;
  flex-direction: column;
  gap: 1rem;
`;
const TwoInputContainer = styled.div`
  display: flex;
  gap: 0.5rem;

  &:last-of-type {
    padding-top: 1rem;
    border-top: 1px solid ${Theme.color.tSeparator};
  }
`;

이렇게 하는걸 의도하신게 아닐까 생각하고 수정해봤어요! 한번 적용하고 확인 부탁드립니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

감사합니다!! 일부 수정하여 반영했습니다:)

colorkite10
colorkite10 previously approved these changes Dec 1, 2023
Copy link
Collaborator

@colorkite10 colorkite10 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 :)

@SongInjae SongInjae dismissed stale reviews from colorkite10 and hyesung99 via 6de192e December 1, 2023 06:23
@SongInjae SongInjae merged commit 078163f into main Dec 1, 2023
2 checks passed
@SongInjae SongInjae deleted the SKRF-447-event-info-qa2 branch December 1, 2023 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants