At this point pretty much copied from InvalidUploadModal with title and children passed in as props. In the next diff will "consume" the Alert component in InvalidUploadModal
Depends on D4206
Details
Details
Will be tested implicitly by the next diff in the stack when we "consume" the newly introduced Alert component from InvalidUploadModal
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/modals/alert.react.js | ||
|---|---|---|
| 16 ↗ | (On Diff #13323) | |
| web/modals/alert.react.js | ||
|---|---|---|
| 12 ↗ | (On Diff #13323) | Maybe we can use something more general? |
| web/modals/alert.react.js | ||
|---|---|---|
| 12 ↗ | (On Diff #13323) | I think it makes sense for this type of Alert component to limit it to text? I fear that opening it up to React.Node could lead to all sorts of layout/styling possibilities if people can pass anything in |