-
Notifications
You must be signed in to change notification settings - Fork 2
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] 라운드 통계 화면 및 게임 결과 화면 구현 #73
+843
−141
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
e61ea09
chore: 동적 파라미터 라우팅 설정 #50
rbgksqkr 0128c04
refactor: BalanceContent 인터페이스 수정 #50
rbgksqkr 752c497
feat: 라우팅 경로 수정 및 라운드 통계 분기처리 #50
rbgksqkr 9650f21
feat: 라운드 통계 수정된 UI 반영 및 Tab UI 구현 #50
rbgksqkr 91f40b3
refactor: 사용하지 않는 파일 삭제 #50
rbgksqkr c65abe3
chore: stylelint css 속성 추가 #50
rbgksqkr aa3b05a
feat: 숫자 카운팅 애니메이션 커스텀 훅 구현 #50
rbgksqkr a200136
feat: roundVoteResult 인터페이스 선언 및 msw 설정 #50
rbgksqkr be3f49a
refactor: 그룹 통계와 전체 통계 서버 상태를 react-query로 관리 #50
rbgksqkr 7103261
refactor: 그룹 통계와 전체 통계 컴포넌트 분리 #50
rbgksqkr 2d26903
refactor: Header 홈 아이콘 대신 현재 라운드 표시 #50
rbgksqkr a04c200
fix: Button disabled 설정 추가 #50
rbgksqkr d8a6d53
feat: 다음 라운드로 이동하는 API 통신 구현 #50
rbgksqkr f46054d
feat: 최종 게임 결과 불러오는 API 구현 #50
rbgksqkr 090e2e4
feat: Header 분기처리 및 라운드 API 불러오기 #50
rbgksqkr 27715e0
feat: 최종 결과 페이지 UI 퍼블리싱 및 API 연결 #50
rbgksqkr 73c9f90
refactor: GameResultItem 컴포넌트 분리 #50
rbgksqkr 93dd3c3
refactor: placeholderData를 이용하여 초기값을 설정하고, 라운드 결과 애니메이션 적용 #50
rbgksqkr 76a259c
chore: console error eslint 룰 추가 #50
rbgksqkr 6e4ec20
refactor: 다음 라운드로 넘어가는 버튼 컴포넌트 분리 #50
rbgksqkr 0a65739
refactor: Tab 컴포넌트 분리 #50
rbgksqkr 261aa4e
refactor: TabContentContainer 컴포넌트 분리 #50
rbgksqkr 347cfb5
refactor: 게임 결과 페이지의 FinalButton 컴포넌트 분리 #50
rbgksqkr adbed42
design: 헤더 아이콘 크기 고정 및 버튼 패딩 제거 #50
rbgksqkr dfc98e4
refactor: 라우팅 경로 상수화 및 헤더 컴포넌트 분기 처리 #50
rbgksqkr 7a2deb5
fix: 컴포넌트명과 타입명이 같아서 생기는 빌드 오류 해결 #50
rbgksqkr 8defa35
style: 카운팅 애니메이션 주석 추가 #50
rbgksqkr 04ffc25
refactor: 라운드 통계 화면 탭과 TabItem 컴포넌트명 수정 #50
rbgksqkr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
style: 카운팅 애니메이션 주석 추가 #50
commit 8defa35fcb6e810434a88b0f435b5f36287d8221
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L5 - 참고 의견
setInterval이 외에도 웹 애니메이션을 최적화하는 requestAnimationFrame 이라는 것도 있더라고요? 저도 이번에 리뷰하면서 알게되었답니다 ㅎ .ㅎ 다음에 최적화하면서 적용해 봐도 좋을 것 같아요 ~!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 저도 숫자 올라갈 때마다 렌더링되는게 문제가 될 수 있다고 생각했는데 requestAnimationFrame 을 적용해볼 수 있겠군요! 나중에 시간이나 성능 비교해보면서 한번에 적용해보면 좋을 것 같아요 ㅎㅎ
https://velog.io/@younghwanjoe/requestAnimationFrame%EC%9D%84-%EC%82%AC%EC%9A%A9%ED%95%98%EC%97%AC-%EC%95%A0%EB%8B%88%EB%A9%94%EC%9D%B4%EC%85%98-%EA%B5%AC%ED%98%84%ED%95%98%EA%B8%B0-%EC%83%81
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
말우 ! 좋은 자료 공유 호라락 읽어보았읍니다 감사룡. . 지금 당장은 이해가 완벽히 안됐지만 역시 해 봐야겠죠 ? 푸하하 최적화 때 야무지게 써 버립시다 ^ .^ ~ ~