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: 고객센터 페이지 퍼블리싱 #81

Merged
merged 7 commits into from
Aug 28, 2023
Merged

feat: 고객센터 페이지 퍼블리싱 #81

merged 7 commits into from
Aug 28, 2023

Conversation

imdaxsz
Copy link
Contributor

@imdaxsz imdaxsz commented Aug 26, 2023

📝 개요

고객센터 페이지 퍼블리싱

helpdesk.mp4

화면 슬라이드 속도가 개발자 도구에서 본 거랑 살짝 다르게 느껴져서 휴대폰으로 테스트한 영상 올립니다.

🚀 변경사항

  • Button, TextInput 컴포넌트에 className?:string 속성 추가
  • Textarea 컴포넌트 추가

사파리에서 input, textarea 클릭 시 위와 같이 화면이 확대되는 현상이 있어서
index.tsx viewport meta 태그에 maximum-scale=1.0, user-scalable=0 추가했습니다.

🔗 관련 이슈

#80

➕ 기타



Figma와 다르게 한 부분들이 있어서 의견을 듣고 싶습니다.

  1. 두 번째 화면에서 보내는 사람, 받는 사람은 배치하니까 뭔가 애매하기도 하고 전체적으로 갑자기 확 딱딱해지는 느낌이 들어서 일단 빼보았습니다. 타 사이트 고객센터 페이지들도 찾아봤는데 대부분 안내 사항과 작성 폼으로만 이루어져 있어서 이 부분은 없어도 될 것 같다는 생각이 듭니다.

  2. 문의 유형은 있어야 할 것 같아서 넣었는데 폰트 사이즈나 배치 등 스타일이 고민이 됩니다. 이쪽 화면의 다른 요소들도 배치가 고민입니다. 의견 주시면 감사하겠습니다.

  3. 제가 올린 영상과 사진에서 보시다시피 폰 세로 화면 크기에 따라 밑에 여백이 꽤 남는 경우도 있는데 현재 요소 배치 및 사이즈가 괜찮은지 의견이 궁금합니다.


@imdaxsz imdaxsz added feature 새 기능 추가 refactor 리팩토링 labels Aug 26, 2023
@imdaxsz imdaxsz requested review from jwun95 and chanwukim August 26, 2023 10:52
@@ -16,6 +16,7 @@ export interface ButtonProps extends ComponentProps<"button"> {
readonly size?: Size;
readonly isBlock?: boolean;
readonly icon?: React.ReactNode;
readonly className?: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

extends ComponentProps<"button">인데 ...props에서 className이 안들어가서 추가된 코드인가요?? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

엇 그 생각을 못 했네요. 불필요한 코드입니다! 삭제하겠습니다~

Comment on lines 7 to 8
${({ warn = false, theme }) => {
return css`
Copy link
Contributor

Choose a reason for hiding this comment

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

화살표 함수니까 { return ...} 제거하면 깔끔해질 것 같아요 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 예전에 작성한 Input 스타일 코드 거의 그대로 가져온 거라 그렇게 됐네요. 두 파일 다 고치겠습니다!

Comment on lines 45 to 46
${({ theme }) => {
return css`
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도 마찬가지입니답


<head>
<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=0" />
Copy link
Contributor

Choose a reason for hiding this comment

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

처음알았네요 배워갑니다 👍

@chanwukim
Copy link
Contributor

두 번째 화면에서 보내는 사람, 받는 사람은 배치하니까 뭔가 애매하기도 하고 전체적으로 갑자기 확 딱딱해지는 느낌이 들어서 일단 빼보았습니다. 타 사이트 고객센터 페이지들도 찾아봤는데 대부분 안내 사항과 작성 폼으로만 이루어져 있어서 이 부분은 없어도 될 것 같다는 생각이 듭니다.

이메일 입력 필드만 [메일아이콘 '답변 받을 메일 주소' ]는 어떨까요?
사용자가 입력하다가 '아 여기 뭐입력했던거더라?'를 아이콘으로 인지할 수 있게요

문의 유형은 있어야 할 것 같아서 넣었는데 폰트 사이즈나 배치 등 스타일이 고민이 됩니다. 이쪽 화면의 다른 요소들도 배치가 고민입니다. 의견 주시면 감사하겠습니다.

추가 했어야 했던 부분인데 감사합니다
아래 이메일 input과 간격을 8px로 줄이면 좋을 것 같아요

제가 올린 영상과 사진에서 보시다시피 폰 세로 화면 크기에 따라 밑에 여백이 꽤 남는 경우도 있는데 현재 요소 배치 및 사이즈가 괜찮은지 의견이 궁금합니다.

괜찮은 것 같습니다

setInquiryContent(e.target.value);
};

const send = () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

정규식은 적용 안하는 걸까요?

let regex = new RegExp('[a-z0-9]+@[a-z]+.[a-z]{2,3}'); 정도만 적용해도 될 것 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이번 PR은 디자인 위주로 진행해서 이후에 추가하려고 합니다!

@imdaxsz
Copy link
Contributor Author

imdaxsz commented Aug 27, 2023

두 번째 화면에서 보내는 사람, 받는 사람은 배치하니까 뭔가 애매하기도 하고 전체적으로 갑자기 확 딱딱해지는 느낌이 들어서 일단 빼보았습니다. 타 사이트 고객센터 페이지들도 찾아봤는데 대부분 안내 사항과 작성 폼으로만 이루어져 있어서 이 부분은 없어도 될 것 같다는 생각이 듭니다.

이메일 입력 필드만 [메일아이콘 '답변 받을 메일 주소' ]는 어떨까요? 사용자가 입력하다가 '아 여기 뭐입력했던거더라?'를 아이콘으로 인지할 수 있게요

문의 유형은 있어야 할 것 같아서 넣었는데 폰트 사이즈나 배치 등 스타일이 고민이 됩니다. 이쪽 화면의 다른 요소들도 배치가 고민입니다. 의견 주시면 감사하겠습니다.

추가 했어야 했던 부분인데 감사합니다 아래 이메일 input과 간격을 8px로 줄이면 좋을 것 같아요

제가 올린 영상과 사진에서 보시다시피 폰 세로 화면 크기에 따라 밑에 여백이 꽤 남는 경우도 있는데 현재 요소 배치 및 사이즈가 괜찮은지 의견이 궁금합니다.

괜찮은 것 같습니다

default.mp4

이렇게 해봤는데 괜찮은가요?

@chanwukim
Copy link
Contributor

  • 영상 상으로는 색이 조금 달라보이네요 input text 컴포넌트랑 똑같은 색이면 될 것 같아요
  • 메일 아이콘 좌측 px을 2px 정도 더주는건 어떄요??

@imdaxsz
Copy link
Contributor Author

imdaxsz commented Aug 27, 2023

  • 영상 상으로는 색이 조금 달라보이네요 input text 컴포넌트랑 똑같은 색이면 될 것 같아요
  • 메일 아이콘 좌측 px을 2px 정도 더주는건 어떄요??

색이 다르다는 게 기본 상태일 때 말씀하시는 걸까요? 현재 기본 색상은 placeholder 및 text와 동일하게 설정했고 focus는 border와 색이 동일합니다. 그리고 이 아이콘 포함한 Input을 HomeDesk에서만 구현을 할까요 아님 기존 TextInput 컴포넌트에 아이콘이 포함될 수 있도록 추가 설정을 해주는 게 좋을까요?

@chanwukim
Copy link
Contributor

blue-60 보다 조금 밝아보여서요! 근데 코드를 보니까 문제가 없네요. 영상에서 밝아보였나봐요
아이콘 추가 설정이 더 범용성있고 좋을 것 같아요 👍

@jwun95 jwun95 self-requested a review August 28, 2023 05:18
@imdaxsz imdaxsz merged commit 43ee388 into develop Aug 28, 2023
@imdaxsz imdaxsz deleted the feature/80 branch August 28, 2023 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새 기능 추가 refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants