DigitalExcellence / dex-frontend

Frontend for the Digital Excellence platform
https://dex.software
GNU Lesser General Public License v3.0
11 stars 6 forks source link

[Bug] Add Project Modal - Ignore Background Click, Add Close Button #595

Closed jrstrayhorn closed 3 years ago

jrstrayhorn commented 3 years ago

Description

On the Add Project Modal, Ignoring background click to close modal (also disabling the use of the Esc button to close modal) and adding a close button in the top right.

Type of change

Checklist

Steps to Test or Reproduce

  1. Add Project
  2. Add project manually
  3. Insert info until step 3
  4. Click outside of modal or hit Esc button, no longer able to close modal.
  5. Click x button in top right to close modal

Photos of Project Model with Close button (desktop and mobile views)

close-button-large

close-button-phone

Video of Close button working https://user-images.githubusercontent.com/21061492/138181076-a92c421d-d832-4d55-98ec-953fdff74f4e.mp4

Link to issue

Closes: #558

jrstrayhorn commented 3 years ago

Hello @BartvanEijkelenburg , @1SvenHansen , @juliaslaats , @MeesvanStraten ,

I've fixed the merge conflict with the changelog.. let me know if there is anything else I need to do on this PR.

Thanks!!