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: 동호회 생성 페이지 구현 #40

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

iamgyu
Copy link
Contributor

@iamgyu iamgyu commented Sep 30, 2024

What is this PR? 🔍

Changes 📝

  • band를 참고하여 동호회 생성 페이지를 구현 하였습니다.

ScreenShot 📷

image

Precaution

  • 이름은 input, 사진은 image, 소개는 textarea를 사용하여 구현 하였습니다.
  • 이미지 선택의 경우 오른쪽에서 버튼을 클릭해 이미지를 설정하면 왼쪽에서 미리보기를 할 수 있도록 구현 하였는데 혹시 다른 의견이 있으시다면 말씀 부탁드립니다!

@iamgyu iamgyu self-assigned this Sep 30, 2024
@iamgyu iamgyu added 5PM🌝 오후 5시 전까지 리뷰가 필요합니다. feat✨ 기능 구현 labels Sep 30, 2024
@Yejin0O0
Copy link
Collaborator

image

p3: 동호회 이름 input에 placeholder가 필요할 것 같습니다.

p4: textarea의 크기가 브라우저에서 조절 가능합니다. 의도하신 부분일까요?

p4: 현재는 파일 첨부하는 영역 안의 '이미지 선택' 글자를 클릭해야지 첨부 가능합니다. 선택 영역이 있는데, 그 안의 특정 요소인 이미지 선택 글자를 눌러야만 첨부 할수 있다는 점이 어색하게 느껴집니다. 이미지 선택 영역을 작게 만들거나, 이미지 선택 영역 자체를 클릭했을 때 파일 첨부가 가능하도록 해야할 것 같습니다.

image p4: 사진을 첨부했을 때 첨부된 사진이 잘려 보이는데, 이는 혹시 의도된 것인가요?

Copy link
Collaborator

@Yejin0O0 Yejin0O0 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@iamgyu
Copy link
Contributor Author

iamgyu commented Sep 30, 2024

p3: 동호회 이름 input에 placeholder가 필요할 것 같습니다.

  • placeholder 추가 완료하였습니다!

p4: textarea의 크기가 브라우저에서 조절 가능합니다. 의도하신 부분일까요?

  • 의도한 부분이 아니기때문에 조절 가능 부분을 삭제하였습니다!

p4: 현재는 파일 첨부하는 영역 안의 '이미지 선택' 글자를 클릭해야지 첨부 가능합니다. 선택 영역이 있는데, 그 안의 특정 요소인 이미지 선택 글자를 눌러야만 첨부 할수 있다는 점이 어색하게 느껴집니다. 이미지 선택 영역을 작게 만들거나, 이미지 선택 영역 자체를 클릭했을 때 파일 첨부가 가능하도록 해야할 것 같습니다.

  • 우선 선택 영역을 지정해놓은 크기 전체로 설정해 놓았습니다! 제 의도로는 글자가 아닌 이미지로 가시성이 더 좋게 하면 어떨까 싶은데 좋은 의견 주시면 좋을 것 같습니다!

p4: 사진을 첨부했을 때 첨부된 사진이 잘려 보이는데, 이는 혹시 의도된 것인가요?

  • 원래는 지정한 크기에 꽉차게 들어가는 것을 원해서 잘려도 그렇게 했지만 내부에 공백이 생겨도 잘리는 부분이 없는게 나을 것 같아서 수정하였습니다!
image

@iamgyu iamgyu closed this Sep 30, 2024
@iamgyu iamgyu reopened this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5PM🌝 오후 5시 전까지 리뷰가 필요합니다. feat✨ 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] Create Club create page (2H)
2 participants