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

chore-fe: Question 타입 Description 추가 및 이에 따른 생성 구현에 대한 로직 수정 #1003

Merged
merged 4 commits into from
Feb 1, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 27, 2025

목적

Question 타입 Description 추가

작업 세부사항

  • Question 타입 Description 추가
  • Question 타입 추가에 따른 생성 로직 변경
  • Qeustion 타입 추가에 따른 수정 로직 변경

참고사항

  • 기본 로직에 description만 추가되었습니다
  • 임의로 title에는 50자, 설명에는 300자 제한을 두었습니다.
image

closes #1002

@github-actions github-actions bot added chore 빌드 업무 수정, 패키지 관리자 구성 등 업데이트, Production Code 변경 없음 frontend 프론트엔드 labels Jan 27, 2025
@lurgi lurgi requested a review from llqqssttyy January 28, 2025 04:00
Copy link
Contributor Author

1738036875.183899

@lurgi lurgi changed the title chore-fe: Question 타입 Description 추가 chore-fe: Question 타입 Description 추가 및 이에 따른 생성 구현에 대한 로직 수정 Jan 28, 2025
@lurgi lurgi marked this pull request as ready for review January 28, 2025 04:02
Copy link
Contributor Author

1738036972.422649

Copy link
Contributor

@llqqssttyy llqqssttyy left a comment

Choose a reason for hiding this comment

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

빠른 작업 감사합니다~ 사용성 관련해서 피드백 하나 남겼으니 확인 부탁드려요!

Comment on lines +111 to +117
<TextField
placeholder="질문 설명을 입력하세요."
onChange={handleChangeDescription}
maxLength={300}
value={description}
isLengthVisible
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

[제안]
현재는 텍스트 필드의 크기를 변경하면 레이아웃이 깨질 수 밖에 없네요.
개선을 위해서는

  • TextField의 resize 옵션에 'horizontal'을 전달하는 방법
  • resize를 false로 전달하고, TextField의 너비, 높이가 부모 요소를 따르도록 변경하는 방법

정도가 생각나네요.
하지만 굳이 고칠 필요 없다 생각되시면 넘어가주세요~
image

Copy link
Contributor

Choose a reason for hiding this comment

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

TextArea 코드 살펴보니 설계의도가 파악되네요. 어떤 말씀하시는지 대강 알겠습니다!
해당 TextArea를 감싸는 container는 flex, direction column으로 동작해서 width 100%처럼 동작하기 때문에 문제될 것은 없다고 생각되네요!

@lurgi lurgi merged commit 574b0b4 into fe/develop Feb 1, 2025
26 of 27 checks passed
@lurgi lurgi deleted the 1002-fe-chore-question-type branch February 1, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 빌드 업무 수정, 패키지 관리자 구성 등 업데이트, Production Code 변경 없음 frontend 프론트엔드
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

2 participants