react-component / react-component/trigger
error of using goNextStatus in useVisibleStatus.ts
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 391
- Fork
- 245
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
Checking goNextStatus() in useVisibleStatus.ts,
the source code look like this, for the switch condition, should we use prev instead of status?
function goNextStatus(callback?: () => void) {
cancelRaf();
rafRef.current = raf(() => {
// Only align should be manually trigger
setStatus((prev) => {
switch (status) { // should we use switch(prev) as it may be possible that react will combine multi setStatus together
case 'align':
return 'motion';
case 'motion':
return 'stable';
default:
}
return prev;
});
callback?.();
});
}
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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 useVisibleStatus.ts và đọc goNextStatus, đặc biệt là updater của setStatus và điều kiện switch của nó. Kiểm tra cách React gộp các cập nhật trạng thái này và liệu giá trị trong closure có thể khác với giá trị prev của updater hay không. Xác nhận tiến trình trạng thái dự kiến và kiểm chứng hành vi bằng các kiểm tra liên quan hiện có trong repository, nếu có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100