[Web] Restore enabled on a full config replace - #4487
Conversation
## Description The web GestureHandler kept the previous enabled value when a full config replace did not carry the key. It now falls back to true, matching Android and iOS. ## Test plan Added a test in `GestureHandler.test.ts`.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe web gesture handler now defaults omitted ChangesGesture enablement
Suggested reviewers: Merge Risk: ⚪ Minimal · up to Web gesture handlers now return to their default enabled state when a replacement configuration omits enabled, matching the intended platform behavior. The focused regression coverage supports merge readiness. 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
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. Comment |
Description
The web GestureHandler kept the previous enabled value when a full config replace did not carry the key. It now falls back to true, matching Android and iOS.
Test plan
Added a test in
GestureHandler.test.ts.