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

Fix/design #80

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix/design #80

wants to merge 2 commits into from

Conversation

KimKwon
Copy link
Member

@KimKwon KimKwon commented Apr 4, 2021

관련 이슈

#72

변경 사항

포스트 업로드 시 UI를 변경했습니다

스크린샷 2021-04-04 오후 6 24 11

스크린샷 2021-04-04 오후 6 24 27

PR Point

디자인에는 소질이 없는 개발자가 한땀한땀 만들어보았습니다
많은 피드백 부탁드립니다.

Reference

@Seogeurim
Copy link
Member

업로드 뷰 너무 이쁜 것 같습니다 !!! 이미지를 올릴 맛이 나네요 너무 예뻐요 ㅎㅎ
그런데 이미지를 / 타이틀을 / 설명을 추가하지 않고 업로드 버튼을 눌렀을 때 프론트에서 에러 메시지를 보여주지 않고 업로드가 되는 것처럼 로딩이 되며 network 탭에서 보면 에러가 잡히고 업로드되지 않습니다 !!
에러메시지에 대한 처리가 있어야 할 것 같아요 !

@KimKwon
Copy link
Member Author

KimKwon commented Apr 19, 2021

업로드 뷰 너무 이쁜 것 같습니다 !!! 이미지를 올릴 맛이 나네요 너무 예뻐요 ㅎㅎ
그런데 이미지를 / 타이틀을 / 설명을 추가하지 않고 업로드 버튼을 눌렀을 때 프론트에서 에러 메시지를 보여주지 않고 업로드가 되는 것처럼 로딩이 되며 network 탭에서 보면 에러가 잡히고 업로드되지 않습니다 !!
에러메시지에 대한 처리가 있어야 할 것 같아요 !

아!! 그 부분을 신경 못 썼네요

  1. title, desc 미입력시 업로드 버튼 비활성화
  2. 미입력 상태로 업로드 버튼 클릭 시 안내메시지

어떤 방식이 더 좋을까요?

@Seogeurim
Copy link
Member

Seogeurim commented Apr 20, 2021

@KimKwon

  1. title, desc 미입력시 업로드 버튼 비활성화
  2. 미입력 상태로 업로드 버튼 클릭 시 안내메시지

어떤 방식이 더 좋을까요?

1번 방식에는 업로드 버튼 비활성화 + 채우지 못한 인풋 부분 빨간색 에러 뜨게 하는거 추가하면 좋을 것 같구
1, 2 중에는 디자인 나온건 없으니 개발하기 더 편한 방식으로 하면 좋을 것 같네용 ㅎㅎ

@Seogeurim
Copy link
Member

@KimKwon 이 부분은 기능 구현 완료되면 언급해서 불러주세요 ~ 바로 리뷰하겠습니당 ~

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