PR Reviewers changes apply when tapping on an empty space

[ismetanin]

Describe the bug
PR Reviewers changes apply when tapping on an empty space.

To Reproduce

1. Go to any PR
2. Click on the floating button
3. Click on the "Reviewers" button
4. Change reviewers list
5. Tap on the empty space outside of the dialog
6. See changes in Reviewers of a PR

Expected behavior
When I tap on the empty space I wanna discard all changes and close dialog. Or maybe do nothing.

Screenshots

Smartphone:

• Device: iPhone 6s
• OS: iOS 12
• Browser Safari
• Version 1.25.0

[rnystrom]

Happy to change the design here so there is an explicit “are you sure?” dialog when cancelling after changes.

_{Sent with GitHawk}

[BasThomas]

I feel that this is a bit trickier, as eg. labels or assignees should then also mimic this behavior. Not sure if it makes sense for labels, however; but having different behavior for different "entities" feels like an even worse user experience. Thoughts?