ScottLogic / prompt-injection

Application which investigates defensive measures against prompt injection attacks on an LLM, with a focus on the exposure of external tools.
MIT License
15 stars 10 forks source link

Give every modal a header with title and close button #782

Closed dhinrichs-scottlogic closed 7 months ago

dhinrichs-scottlogic commented 7 months ago

Feature Request

Description

Acceptance criteria

GIVEN A modal is open THEN there is a header with title and close button

WHEN A visual user looks for the close button THEN they see a "Close" button and an X icon with a round border,

WHEN A screen reader user tabs to the close button THEN they hear "Close, button" but do not hear the icon announced.

Wireframes:

Welcome modal: image Mission Info: image Document Viewer: image Handbook: image

Notes:

Who knew close buttons can be so controversial 😄! Here's some interesting articles on this: https://www.htmhell.dev/20-close-buttons/ https://statics.teams.cdn.office.net/evergreen-assets/safelinks/1/atp-safelinks.html

m4rkwallace-SL commented 7 months ago

Added these to the figma file

Screenshot 2024-01-22 at 09 51 55 Screenshot 2024-01-22 at 09 51 17
dhinrichs-scottlogic commented 7 months ago

During discussion, we anticipated an issue with the scrolling of the modals interfering with the close button. One idea to work around that was to give the modals headers, with a heading and a close button. Headings would be a great addition to the modals. image