fix: destructive modal close behavior#1138
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR adds the ChangesModal Outside-Click Prevention Configuration
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
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 |
Pre-Submission PR Checklist
Description of the change
Fixes modal escape behavior on destructive actions like bulk user actions, and user transfers. Users can now only exit the modal through the x button, cancel, and escape key.
BulkActionDialogs.tsx — BulkResetPassword (L150), BulkDeactivate (L328), BulkDelete (L474)
ResidentDialogs.tsx — Deactivate (L344), Delete (L470), Transfer (L589)
DeleteClassModal.tsx (L50)
Related issues: https://app.asana.com/1/1201607307149189/project/1209460078641109/task/1215131538607872?focus=true