Skip to content

Fix/modal a11y - #104

Merged
Lakes41 merged 3 commits into
Adamantine-guild:mainfrom
jchinedu:fix/modal-a11y
Jul 29, 2026
Merged

Fix/modal a11y#104
Lakes41 merged 3 commits into
Adamantine-guild:mainfrom
jchinedu:fix/modal-a11y

Conversation

@jchinedu

Copy link
Copy Markdown
Contributor

Description

Briefly describe the changes made in this pull request.

Linked Issue

Closes #[issue number]

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • UI/UX improvement
  • Performance improvement
  • Code refactoring

Summary of Changes

  • Change 1
  • Change 2
  • Change 3

Test Evidence

Describe how you tested these changes:

# Test commands run
npm run lint
npm run typecheck
npm run build

Include screenshots or recordings if relevant for UI changes.

Accessibility Considerations

Describe any accessibility implications:

  • Screen reader compatibility
  • Keyboard navigation
  • Color contrast
  • ARIA labels

Checklist

  • My code follows the project's style guidelines (React, TypeScript, Tailwind)
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have updated the documentation (README, component comments)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally
  • I have run npm run lint and there are no linting errors
  • I have run npm run typecheck and there are no type errors
  • I have built the project with npm run build successfully
  • These changes do not break backwards compatibility (or I have documented the breaking change)
  • I have considered accessibility implications

Additional Context

Add any other context, screenshots, or recordings about the pull request here.

Reviewer Notes

Any specific areas you'd like reviewers to focus on (UX, accessibility, performance, etc.).

@Lakes41
Lakes41 merged commit b5ce6e2 into Adamantine-guild:main Jul 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants