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

여러 페이지 보완 #180

Merged
merged 16 commits into from
Feb 10, 2022
Merged

여러 페이지 보완 #180

merged 16 commits into from
Feb 10, 2022

Conversation

100Gyeon
Copy link
Member

@100Gyeon 100Gyeon commented Feb 9, 2022

⛓ Related Issues

📋 작업 내용

1. 피드백 생성 페이지

  • 피드백이 없을 때 나타날 empty view 추가
  • 피드백, 키워드 입력 완료하기 전 버튼 비활성화 처리

2. 이슈 등록 페이지

  • 이슈 카테고리 선택하는 곳, 팀에서 겪은 상황 작성하는 곳 분리
  • 전반적인 css 수정

3. 너가소개서 결과 페이지

  • 답변이 없을 때 나타날 empty view 수정
  • 답변이 없을 때도 키워드 모음 보이는 문제 해결

📌 PR Point

  • 피드백 생성 페이지만 보완하려고 했으나 다른 페이지에서도 비슷한 점들을 보완해야 해서 같이 수정했습니다.
  • 컨벤션에 어긋나는 이미지 파일명, 적절하지 않은 파일명이 있어 수정했습니다.
  • chore: 이슈 등록 페이지 모달 텍스트 수정이라고 남긴 커밋이 있는데 이슈 등록 페이지가 아니고 피드백 생성 페이지입니다.
  • 여러 페이지에 UX Writing 수정사항 반영했습니다.

👀 스크린샷 / GIF / 링크

1. 피드백 생성 페이지

image

2. 이슈 등록 페이지

image

3. 너가소개서 결과 페이지

image

@100Gyeon 100Gyeon added fix ☔️ 에러 고치기 refactor 🪐 리팩토링 chore 🍣 기능 개발 외 부수적인 일들 labels Feb 9, 2022
@100Gyeon 100Gyeon self-assigned this Feb 9, 2022
@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@Hyoin-Kim
Copy link
Member

나도 리뷰어 달아줘,,,,,,, 회사에서 볼래,,,,,,,,,

@100Gyeon
Copy link
Member Author

@Hyoin-Kim 우악 효인언니다 !!!! 좋아 ❤

@100Gyeon 100Gyeon requested a review from Hyoin-Kim February 10, 2022 05:21
Copy link
Member

@NamJwong NamJwong left a comment

Choose a reason for hiding this comment

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

꼼꼼지연 고생했어요 !!

@@ -0,0 +1,21 @@

Copy link
Member

Choose a reason for hiding this comment

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

따로 파일 분리해서 컴포넌트 만든 거 좋아요!!

Copy link
Member Author

Choose a reason for hiding this comment

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

서진이가 Empty 폴더 만들어서 작업한 게 있길래 나도 여기에 만들었어 ㅎㅎ

width="100%"
placeholder="팀원에게 이슈에 대한 피드백을 남겨주세요"
disabled={true}
/>
</StLink>
<Outlet />
Copy link
Member

Choose a reason for hiding this comment

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

헉 이거 이렇게 처리한지 처음 알았다,, 궁금했는데 알게된!

Copy link
Member

Choose a reason for hiding this comment

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

(Outlet 말한거)

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

@SeojinSeojin
Copy link
Member

울 웹쁜이 고생많았어 ! 여기서 미리보기로 보면서 쉬어~ 다른 웹쁜이들한테도 자랑해줘~

Copy link
Member

@Hyoin-Kim Hyoin-Kim left a comment

Choose a reason for hiding this comment

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

디자인이 좀 바꼈구나,,!! 피그마 들어가서 나도 확인해봐야겠다!!
내부분인데 해줘서 너무 고마워 박박지여니,, 고생많았어~

Copy link
Member

@SeojinSeojin SeojinSeojin left a comment

Choose a reason for hiding this comment

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

박박.... 넘 꼼꼼하게 잘했다ㅜㅜㅜㅜ💖

@@ -17,7 +17,7 @@ function IssueMemberList(props: IssueMemberListProps) {
<StIssueMemberList>
<div>
{issueMembers.slice(0, MAX_IMAGE_NUM).map(({ id, profileImage }) => (
<img key={id} src={profileImage ?? imgEmptyProfile} />
<img key={id} src={profileImage || imgEmptyProfile} />
Copy link
Member

Choose a reason for hiding this comment

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

이렇게 하면 머 나오나여!?

Copy link
Member Author

Choose a reason for hiding this comment

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

image

?? 쓰면 이렇게 나와서 ||로 수정했어! ㅠㅠ

Copy link
Member

Choose a reason for hiding this comment

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

넘웃기구 어이업서 ㅜㅜㅜㅜㅜㅜㅜ 고마워유!!!

@100Gyeon 100Gyeon merged commit ade5d28 into dev Feb 10, 2022
@100Gyeon 100Gyeon deleted the feat/#179 branch February 10, 2022 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 🍣 기능 개발 외 부수적인 일들 fix ☔️ 에러 고치기 refactor 🪐 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

피드백 생성 페이지 보완
4 participants