NativeScript / NativeScript/angular
Modal route issue - route runs twice when returning from second modal containing route outlet
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 55
- Fork
- 13
- Merge trung bình
- 16 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
This is most evident in iOS. I've set up a demo app:
https://stackblitz.com/edit/nativescript-stackblitz-templates-vwcmqt
In iOS, if you follow the steps (they're numbered) you'll see that when you navigate back on step five, the animation runs twice—backwards then forwards. This appears to come from the shouldDetach method NSRouteReuseStrategy. If I use the previous code for shouldDetatch that was altered here:
https://github.com/NativeScript/angular/commit/bebcde23c4d6a392638c23ec1e2bc63b06a9b43e
then it works without problem.
NOTE: For this demo, I've used 14.2.6-alpha.0 because without it, I can't use modals with ModalDialogParams because of injector scope issues. However, this router problem described above also occurs in 14.2.5 when the modal corrections are in place. Either way, something isn't working as it should.
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 phương thức NSRouteReuseStrategy.shouldDetach và so sánh nó với phần triển khai trước đó trong commit beb cde23c4d6a392638c23ec1e2bc63b06a9b43e. Tái hiện luồng được đánh số trong bản demo StackBlitz được liên kết trên iOS, bao gồm cả việc quay lại từ modal thứ hai có chứa một route outlet. Hoàn thành khi thao tác điều hướng quay lại ở bước năm chạy animation một lần thay vì chạy ngược rồi chạy xuôi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- angular, typescript
- Lĩnh vực
- frontend, mobile-dev
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- 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
- 35/100