Better implementation of render props
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- JavaScript
- Star
- 11.8k
- Fork
- 7.9k
- Merge trung bình
- 1 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 11
Mô tả
This page https://reactjs.org/docs/render-props.html introduces the "render props" pattern with a class Mouse. Its implementation is
render() {
return (
<div>
<h1>Move the mouse around! 1</h1>
<Mouse render={mouse => <Cat mouse={mouse} />} />
</div>
);
}
And its usage is
<Mouse render={mouse => <Cat mouse={mouse} />} />
I find that if we implement it as
render() {
const { render: Render } = this.props;
return (
<div
style={{ height: 100, border: "1px solid" }}
onMouseMove={this.handleMouseMove}
>
<Render mouse={this.state} />
</div>
);
}
Then we can allow the following usage patterns:
<Mouse render={({ mouse }) => <Cat mouse={mouse} />} /> // the original Cat class, 1st pattern
<Mouse render={MediumCat} /> // MediumCat is a function component, 2nd pattern
<Mouse render={BigCat} /> // BigCat is a class component, 3rd pattern
With this implementation, we have a pattern (the first one) which is almost the same as the original render props. Also it allows another two usage patterns (actually the 1st and 2nd patterns are the same).
I have created a codesandbox demo. In the demo, MouseTracker1 is the same as documented while MouseTracker2 is different in implementation of Mouse as I suggested.
I wonder if this is a better implementation of "render props". Any thought?
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với trang render-props được liên kết trong issue và bản demo CodeSandbox đi kèm. So sánh cách triển khai Mouse được ghi trong tài liệu với biến thể được đề xuất; được xem là hoàn tất khi một maintainer quyết định liệu tài liệu có nên thay đổi hay không, đồng thời thống nhất phạm vi cho bất kỳ bản cập nhật nào.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript, react
- Lĩnh vực
- documentation
- Loại issue
- Tài liệu
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 20/100