SimplyStaking / panic

PANIC Monitoring and Alerting For Blockchains
Apache License 2.0
82 stars 31 forks source link

Every modal must be non-closable when clicking on background #335

Closed dillu24 closed 1 year ago

dillu24 commented 1 year ago

Technical Story

As a user I want that my configurations are not lost when I accidently click on the modal background

Requirements

Blocked by

336

Acceptance criteria

Given: Any Modal When: I click on the background Then: The modal is not closed

itsciccio commented 1 year ago

I suggest going through the entire PANIC UI and applying this fix to each modal, mainly those that include user input (forms).

itsciccio commented 1 year ago

We also need to investigate the case of when the user hits ESC on their keyboard. After implementing the fix that this ticket covers, please test if the modal still closes upon ESC is hit on the keyboard. If closes, we need to investigate from the UI Kit side.