theforeman / foreman / 16987

64%

push

travis-ci

Fixes #27794: Add ForemanModal

add basic ForemanModal component based on patternfly-react Modal
wip composable header/footers
separate out Header and Footer subcomponents
add Hooks and Context to pass data to subcomponents
add several Storybook stories
move aside outside the Modal
add new story for props
refactor after laviro review
allow modal hide using props instead of not rendering
Add snapshot tests
fix linter i hope
more refactor after review
Add unit tests
add close button tests
switch from lodash omit to native JS
extract extractModalNodes to helper function
Don't pass title prop as it causes unwanted tooltip
rename file to ForemanModalHooks.js
update import
move useModalState to stories
use arrow function over regular function
fix lint issue
add esdoc comments to helper
add test for helper
extract context into its own file to avoid circular imports

remove title from PropTypes
update snapshots
fix linter

1422 of 2319 branches covered (61.32%)

4155 of 6050 relevant lines covered (68.68%)

43.64 hits per line