Skip to content

Create Modal Component#165

Merged
willbreitkreutz merged 12 commits intomainfrom
158-modal
Mar 26, 2025
Merged

Create Modal Component#165
willbreitkreutz merged 12 commits intomainfrom
158-modal

Conversation

@krowvin
Copy link
Contributor

@krowvin krowvin commented Feb 27, 2025

Solves

This PR

  1. Adds a Modal component that wraps the headlessui Dialog component and provides a standard template with various properties you can provide to control it.
  2. Adds documentation for the Modal component
  3. Converts the DoD warning to use this new Modal component

@USACE USACE deleted a comment from github-actions bot Feb 27, 2025
@krowvin krowvin linked an issue Feb 27, 2025 that may be closed by this pull request
@willbreitkreutz
Copy link
Contributor

@krowvin can you resolve conflicts and i can merge

@krowvin
Copy link
Contributor Author

krowvin commented Mar 11, 2025

@willbreitkreutz merge conflicts resolved

@willbreitkreutz willbreitkreutz self-requested a review March 26, 2025 14:19
@willbreitkreutz willbreitkreutz merged commit 9f458ba into main Mar 26, 2025
@willbreitkreutz willbreitkreutz deleted the 158-modal branch March 26, 2025 14:19
krowvin added a commit that referenced this pull request Mar 31, 2025
* Add Modal to Library

* Center modal title and ensure Description is from correct library

* Convert dod-warning to Modal component for extra dog food 🐶

* Create modal documentation

* Add a warning for if a user provides a size that does not exist

* Centralize width options for re-use

* Remove unused classes

* Make the modal more interesting

* Remove responsive variants from width options

* Remove debug statements

* Cleanup onClick for refresh button
krowvin added a commit that referenced this pull request Mar 31, 2025
* Add Modal to Library

* Center modal title and ensure Description is from correct library

* Convert dod-warning to Modal component for extra dog food 🐶

* Create modal documentation

* Add a warning for if a user provides a size that does not exist

* Centralize width options for re-use

* Remove unused classes

* Make the modal more interesting

* Remove responsive variants from width options

* Remove debug statements

* Cleanup onClick for refresh button
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Modal Component

2 participants