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.

Sign up for GitHub

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

Jump to bottom

[박상범] 챕터 12, 13 #104

Merged
merged 2 commits into from
Dec 20, 2024
Merged

[박상범] 챕터 12, 13 #104

merged 2 commits into from
Dec 20, 2024

Conversation

sangbooom
Copy link
Contributor

No description provided.

@sangbooom sangbooom self-assigned this Dec 2, 2024
hyesungoh
hyesungoh approved these changes Dec 4, 2024
View reviewed changes
챕터_12/상범.md
Comment on lines +9 to +12
### 단점
- prop 이름 충돌 가능성 => 어떤 고차 컴포넌트가 어떤 prop 제공하는지 파악하기 어려워 디버깅이 어려울 수 있음

> 이것만 단점으로 꼽는다고?! 복잡성과 트리 깊이 문제가 단점이라 생각
Copy link
Member

Choose a reason for hiding this comment

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

👍 👍 ㅇㅈ합니다

챕터_12/상범.md
Comment on lines +90 to +95
## 리스트 가상화
한번에 수천개의 목록 요소를 렌더링하여 초기 렌더링 속도 저하 또는 스코롤 성능 저하를 유발하는 대신 가상화는 사용자에게 보이는 항목만 렌더링 시킴

> 실제로 무한 스크롤로 이루어진 페이지에서 이미지와 데이터가 DOM에 계속 쌓이다보면 점점 느려지는 경우가 있음
> 무신사는 virtuoso, 오늘의집은 예전에는 react-virtualized 쓰다가 최근에 virtuoso로 갈아탄듯함 [오늘의집 무한스크롤 개발기](https://www.bucketplace.com/post/2024-09-11-virtualizedlist-%EB%AC%B4%ED%95%9C%EC%8A%A4%ED%81%AC%EB%A1%A4-%EB%A6%AC%EC%8A%A4%ED%8A%B8-%EB%A6%AC%ED%8C%A9%ED%86%A0%EB%A7%81-%EA%B0%9C%EB%B0%9C-%EC%9D%B4%EC%95%BC%EA%B8%B0/)

Copy link
Member

Choose a reason for hiding this comment

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

갓갓갓 ㄷㄷㄷㄷ

챕터_13/상범.md
Comment on lines +36 to +39

streaming ssr 공부할 때 좋았던 레퍼런스
> [SSR의 기쁨과 슬픔: 렌더링의 변화의 흐름을 통해 알아보는 SSR과 Streaming SSR | 인프콘2023](https://www.youtube.com/watch?v=hPyyFu3lrEg&t=2080s)
> [[Dev Dive_ Frontend Day] 스트리밍 SSR 딥 다이브](https://www.youtube.com/watch?v=9xl9X2pfHeI&t=1835s)
Copy link
Member

Choose a reason for hiding this comment

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

최고

@sangbooom sangbooom merged commit e1cbc11 into main Dec 20, 2024
1 check passed
@sangbooom sangbooom deleted the 챕터12,13/박상범 branch December 20, 2024 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyesungoh hyesungoh hyesungoh approved these changes

@sjoleee sjoleee Awaiting requested review from sjoleee sjoleee is a code owner

@Seojunhwan Seojunhwan Awaiting requested review from Seojunhwan Seojunhwan is a code owner

@metacode22 metacode22 Awaiting requested review from metacode22 metacode22 is a code owner

@Orchemi Orchemi Awaiting requested review from Orchemi Orchemi is a code owner

@sumi-0011 sumi-0011 Awaiting requested review from sumi-0011 sumi-0011 is a code owner

@100Gyeon 100Gyeon Awaiting requested review from 100Gyeon 100Gyeon is a code owner

@WooWan WooWan Awaiting requested review from WooWan WooWan is a code owner

Assignees

@sangbooom sangbooom

Labels
챕터_12 챕터_13
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sangbooom @hyesungoh