topcoderinc / TC-redis-manager

5 stars 15 forks source link

"Add New Redis Server" modal should not be dismissed if the error message is shown #358

Open winterflame opened 5 years ago

winterflame commented 5 years ago

Reproduction Steps

  1. deploy with default test data and open the app

  2. click "+"

  3. click "Add" button on the modal

Expected Result

"Add New Redis Server" modal should not be closed if the error message is shown, so that user can correct the input on the modal.

Actual Result

See screenshot/screencast, the modal will be closed when the error message is shown.

Screenshots or screencast

If not attached, please check comment section.

Issue Information

Device: Mac Book Pro

OS: macOS High Sierra Version 10.14

Browser: Safari 12.0

winterflame commented 5 years ago

error

User will need to click "+" again currently.

modal