-
-
Notifications
You must be signed in to change notification settings - Fork 187
Add Hook for managing SnapshotController instances #464
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
Add Hook for managing SnapshotController instances #464
Conversation
WalkthroughAdds a new snapshot controller hook: a part directive in Changes
Sequence Diagram(s)sequenceDiagram
participant Widget as Widget Tree
participant HookBuilder as HookBuilder
participant Hook as useSnapshotController
participant Controller as SnapshotController
Widget ->> HookBuilder: build
HookBuilder ->> Hook: call useSnapshotController(allowSnapshotting)
Hook ->> Controller: create SnapshotController(allowSnapshotting)
HookBuilder -->> Widget: render with Controller
Note over Hook,Controller: On hook updates
Hook ->> Controller: update allowSnapshotting
Note over Widget,Hook: On widget disposal
Widget ->> Hook: dispose hook
Hook ->> Controller: dispose Controller
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
…into pr/benthillerkus/464
Adds a
useSnapshotController
hook for https://api.flutter.dev/flutter/widgets/SnapshotController-class.htmlSummary by CodeRabbit
New Features
Tests
Documentation