Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
anikbrazeau opened this issue Nov 28, 2024 · 2 comments
Labels
App bug Something isn't working

Comments

@anikbrazeau
Copy link
Member

anikbrazeau commented Nov 28, 2024

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

  • Browser: Edge
  • Operating system: Windows
  • Environment: Staging
  • Language: English
  • Viewport: Tablet

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

@anikbrazeau anikbrazeau added App bug Something isn't working labels Nov 28, 2024
@anikbrazeau anikbrazeau changed the title Need to scroll to see top/bottom of modal with key actions [API v2 QA] Need to scroll to see top/bottom of modal with important user actions [API v2 QA] Nov 28, 2024
@timarney
Copy link
Member

timarney commented Dec 3, 2024

@Abi-Nada

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

@anikbrazeau anikbrazeau changed the title Need to scroll to see top/bottom of modal with important user actions [API v2 QA] Need to scroll to see top/bottom of modal with important user actions Dec 3, 2024
@anikbrazeau anikbrazeau changed the title Need to scroll to see top/bottom of modal with important user actions Modal design: Need to scroll to see top/bottom of modal with important user actions Dec 3, 2024
@Abi-Nada
Copy link

Abi-Nada commented Dec 4, 2024

Makes sense! Moving it out of Sprint Backlog so we can prioritize API-specific work accordingly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants