Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 2 Current »

All public Working Group meetings follow the Recording Policy for Open edX Meetings

\uD83D\uDDD3 Date

\uD83D\uDC65 Participants

\uD83E\uDD45 Goals

\uD83D\uDDE3 Discussion topics

Time

Item

Presenter

Notes

Toast contents

Adam Stankiewicz on behalf of Braden MacDonald and Rômulo Penido

Currently, Toast only accepts a string as its contents (i.e., children) per the PropType definition.

An open PR is proposing to update the PropType to allow any node, rather than only strings.

The given use case was primarily to be able to pass FormattedMessage (from react-intl), but this could also be done through intl.formatMessage to appease the existing PropType.string.

The Toast does accept an optional action prop, which renders an appropriately styled Button within the Toast.

[question] Perhaps the reason it’s a string is to help indicate that nodes such a CTAs/hyperlinks should be done through the action prop?

[question] Is there any reason we wouldn’t want to allow any node in the Toast contents, to support things like lists, etc.?

image-20240918-145026.png





✅ Action items

  •  

⤴ Decisions

  • No labels