search

cds-snc / platform-forms-client

NextJS application that serves the public-facing website for Forms
https://forms-staging.cdssandbox.xyz/
MIT License
35 stars 13 forks source link

Modal design: Need to scroll to see top/bottom of modal with important user actions #4728

Open anikbrazeau opened 6 days ago

anikbrazeau commented 6 days ago

Description

In long modals, the need to scroll obfuscates the heading/close button and the CTA buttons.

Steps to reproduce

  1. Go to a draft form
  2. Select API delivery in Settings
  3. Create an API key
  4. See need to scroll in modal
  5. Test out filling out a response
  6. Go to responses page
  7. Select download to open another modal and see the scroll hide the CTA buttons

Details

Expected behaviour

Consider freezing the frame with the heading/title/close button statically locked and the CTAs also always visible

Screensh

scroll no scroll ots or videos

timarney commented 1 day ago

@Abi-Nada

Perhaps we look at this card for Dialogs in general vs the API ... we re-use the dialog across the app.