You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In https://oui.opensearch.org/1.0/#/layout/modal, we have a section about how to manually specify widths via inline styles. This seems like a bit of an antipattern, and not something we want to encourage. See #783 for examples where we do customize modal widths today.
If we do want to allow direct sizing controls, we probably want to add more props. Otherwise, I think we should just scratch the section altogether.
The text was updated successfully, but these errors were encountered:
I agree that we should remove this guidance - it does not align with the Design System directive to rely on the responsive system as much as possible out of the box.
BSFishy
changed the title
Should we remove the "Widths" section from the OuiModal docs?
Remove the "Widths" section from the OuiModal docs
Jun 20, 2023
In https://oui.opensearch.org/1.0/#/layout/modal, we have a section about how to manually specify widths via inline styles. This seems like a bit of an antipattern, and not something we want to encourage. See #783 for examples where we do customize modal widths today.
If we do want to allow direct sizing controls, we probably want to add more props. Otherwise, I think we should just scratch the section altogether.
The text was updated successfully, but these errors were encountered: