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: ProgressIndicator 컴포넌트 #60

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Andrevile
Copy link
Collaborator

🎉 변경 사항

스크린샷 2024-07-18 오전 2 25 28

🔗 링크

🙏 여기는 꼭 봐주세요!

호되게 맞을 각오하고 올립니다.

Copy link
Collaborator

@semnil5202 semnil5202 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 👍👍 제가 보기엔 충분히 잘 구현된 것 같습니다.

몇 가지 궁금한 부분 남겨두었습니다.

</div>
);
},
) as React.ForwardRefExoticComponent<ProgressIndicatorProps> & { Step: typeof Step };
Copy link
Collaborator

Choose a reason for hiding this comment

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

ForwardRefExoticComponent 라는 타입이란걸 처음 알았네요. 최근에 ref 때문에 다형성 컴포넌트 타입 추론 오류를 겪었는데 이게 해결책이 되겠군요.

@Doeunnkimm 아래 포스팅 보시면 Txt 다형성 컴포넌트 타입 추론 문제를 해결할 수 있을 것 같습니다.

Polymorphic한 React 컴포넌트 만들기

color = '#FF7664';
}

return <span {...rest} css={stepCss({ basis, color })} />;
Copy link
Collaborator

Choose a reason for hiding this comment

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

단순한 궁금증인데 span 태그로 하신 이유가 있을까요??

return <span {...rest} css={stepCss({ basis, color })} />;
};

ProgressIndicator.displayName = 'ProgressIndicator';
Copy link
Collaborator

Choose a reason for hiding this comment

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

요 친구는 무슨 용도인가요??

@Doeunnkimm
Copy link
Member

@semnil5202 아 이거 종오님이 Txt랑 파일 컨벤션 맞춰서 핑 주신다고 하셔서 코드 변경될 수도 있서용

semnil5202 added a commit that referenced this pull request Jul 29, 2024
semnil5202 added a commit that referenced this pull request Aug 8, 2024
semnil5202 added a commit that referenced this pull request Aug 14, 2024
semnil5202 added a commit that referenced this pull request Aug 17, 2024
* chore: relay start assets 세팅

* chore: next.js 의존성 전체로 지정

* feat: 릴레이 질문 시작 Profile 구현

* design: Profile Style 수정

* feat: 릴레이 질문 시작 페이지 구현

* feat: CreateRelayQuestionScreen page.tsx로 연동

* rename: relay-question으로 도메인명 변경

* rename: create->start로 리네임

* feat: select relay question 페이지 생성

* feat: select relay page로 라우팅 기능 추가

* feat: ProgressIndicator clone

#60

* rename: Relay 오타 수정

* feat: 릴레이 질문 리스트 나열 기능 추가

* rename: 폴더 구조 및 css 변수명 수정

* feat: 질문 상세 모달 콘텐츠 생성

* feat: query string context 생성

* refactor: Question List 하단 여백 보장

* feat: Progress Indicator Container 추가

* feat: 랜덤 클릭 버튼 추가

* feat: ToolTip 생성 및 5초간 생성 후 삭제 기능 추가

* rename: Container renaming

* feat: Questioner 생성 및 모달 오픈 로직 hook 분리

* feat: 다음 질문인 선택시 모달 표시

* feat: 다음 질문인 선택 후 그룹 공유 페이지로 라우팅 기능 추가

* refactor: Random 버튼 클릭 시에만 랜덤 다시선택 텍스트 표시토록  변경

* feat: share-group background 및 currentQuestion Count 구현

* refactor: css 네이밍 수정 및 적용

* feat: 질문 생성 완료 및 그룹에게 공유 페이지 구현

* feat: next Questioner로 라우팅하는 기능 추가

* refactor: 스타일링 수정

* feat: 다음 릴레이 질문자 선택 페이지 구현

* refactor: css, rest prop 누락 수정

* refactor: screen async 추가

* fix: add suspense useSearchParams

* fix: suspense fix

* fix: suspense err fix

* fix: suspense err fix - 2

* fix: suspense err fix - 3

* fix: suspense err fix - 4

* refactor: prefetch 적용 - start-relay-question 파트

* feat: query 훅 구현 - select relay question

* design: minHeight 100dvh로 임시 변경

* refactor: mock 데이터 이미지 추가

* chore: use-overlay 설치

* feat: start-relay-question api end point change

* chore: next config moring image url add

* refactor: useModal refactoring using toss-overlay

* refactor: prefetch select relay question screen

* refactor: apply toss-overlay useModal

* feat: implement useRandomQuestionQuery

* design: css 수정

* refactor: 질문 서버 데이터 연동 및 temp 로직 분리

* design: edit css

* refactor: separate RePick Component

* refactor: 랜덤 다시 선택 api 연동

* design: style css refactoring

* feat: implement infinite query and question list intersection

* feat: feature meeting members fetching

* design: edit svg position

* refactor: refactoring MyInfo types and css

* refactor: remove useQueryString and type direct

* refactor: use my info api to ContentContainer title

* design: refactoring question component style

* feat: make useActiveQuestion when it used share-group

* refactor: refactoring query string and data fetching

* fix: size token 2xl value fix

* refactor: fix prefetching error temporary

* design: share-group svg container width refactoring

* feat: 카카오 및 링크 복사 모달 콘텐츠 생성

* feat: 공유하기 모달 구현

* rename: useModal 디렉토리 위치 변경

* feat: Toast 기능 구현

* refactor: useSuspenseQuery -> useQuery change

* fix: add query meetingId dependencies

* refactor: query key 보완

* refactor: copy clipboard logic refactoring

* refactor: Toast 기본 노출 시간 변경

* feat: 카카오톡 공유하기 기능 추가

* feat: 카카오톡 공유하기 기능 적용 및 관련 로직 수정

* fix: window location 적용

* refactor: rebase 이후 lint 에러 수정

* chore: eslint import order error fix

* feat: Kakao Script 주입

* refactor: 임시로 라우팅한 로직 복원

* refactor: 페이지 export 문 변경

* chore: gitignore 추가

* refactor: prefetch

* fix: undefined window 문제 해결

* refactor: gitignore certificates 폴더 위치 변경

* refactor: SelectRelayQuestion prefetching disable

* remove: KakaoScript 중복 제거

* feat: getWebDomain 유틸 함수 및 process env Turbo 설정

* feat: 제일 먼저 답변하기 URL 라우팅 연결 추가

* refactor: meetings API 스펙 변경 사항 적용

* refactor: 카카오톡 공유하기 모달 common 것으로 적용

* refactor: URLSearchParams -> useSearchParams로 일괄 변경

* refactor: indicator focus color 누적 형식으로 변경

* refactor: QuestionDetail 질문 미리보기 정렬 css 수정

* feat: 모임에서 내 id 조회 query 추가 및 일부 query 로직 보완

* refactor: action bar 추가 및 적용

* remove: 미사용 Hook 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants