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

[Contacts articles] Contacts 페이지(1/2) - 연락처 컴포넌트 구현 #4

Merged
merged 12 commits into from
Sep 28, 2023

Conversation

Doyu-Lee
Copy link
Owner

@Doyu-Lee Doyu-Lee commented Sep 28, 2023

구현 기능

한국어 페이지 | 영어 페이지

연락컴포최종

1. 로딩 컴포넌트로 폰트 등의 사용자 경험 향상 시도

바로 직전 PR #3 에서 한글 폰트 최적화가 안 되는 이슈 발생

  • 헤더에 로딩 컴포넌트를 넣어두고 로딩, 실제 화면 컴포넌트를 동시에 렌더링한 후 0.8초 뒤에 로딩 컴포넌트가 사라지면서 렌더링된 화면을 보여줄 수 있게 구현함

2. 타이핑 효과 애니메이션 컴포넌트 및 영/한 데이터 추가

  • react-typist 라이브러리를 사용했지만 componentWillReceiveProps has been renamed, and is not recommended for use. See https://reactjs.org/link/unsafe-component-lifecycles for details.와 같이 최신 React 18버전 이상에서 호환되지 않는 일부 성능 문제가 발생
  • 해당 라이브러리 레포지토리 이슈에서 관련 문제 발견 후, 2022년 초부터 업데이트가 안 되고 있다는 것을 확인
  • react-simple-typist로 라이브러리 교체 후 이상없이 작동

3. hook 생성하여 코드 분리

  • useAfterSeconds(n) : n밀리세컨드초(ms) 뒤에 isLoading 또는 isMounting 상태 변경
  • useHover() : 로고 마우스 hover했을 때 애니메이션 hoook

4. 스타일 전역 변수 및 mixin 정리

  • scss 전역 적용되는 _mixins.scss 파일에서 중복해서 쓰이는 스타일 정리
  • 폰트 스타일, 모서리 둥글기 전역 변수로 추가

@Doyu-Lee Doyu-Lee added enhancement New feature or request UI feature labels Sep 28, 2023
@Doyu-Lee Doyu-Lee self-assigned this Sep 28, 2023
@vercel
Copy link

vercel bot commented Sep 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
portfolio-doyu ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 28, 2023 2:48am

@Doyu-Lee Doyu-Lee merged commit 112dbd6 into dev Sep 28, 2023
1 check passed
@Doyu-Lee Doyu-Lee deleted the contacts-articles branch October 12, 2023 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant