agw/overleaf-cep
1
0
Fork 0
mirror of https://github.com/yu-i-i/overleaf-cep.git synced 2026-05-25 10:10:08 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #14793 from overleaf/jel-new-alert-content-string

Browse Source 
[web] Option to pass content as string to notification component

GitOrigin-RevId: 05196a44d25dab5ba85b58965c3bb5ac071a3897
This commit is contained in:
Jessica Lawshe
2023-09-13 07:33:14 -05:00
committed by Copybot
parent 04a803e8e9
commit 1fd436675b
3 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/web/frontend/js/shared/components/notification.tsx
View File

@@ -8,7 +8,7 @@ type NotificationType = 'info' | 'success' | 'warning' | 'error'
type NotificationProps = {
action?: React.ReactElement
ariaLive?: 'polite' | 'off' | 'assertive'
content: React.ReactElement
content: React.ReactElement | string
customIcon?: React.ReactElement
isDismissible?: boolean
isActionBelowContent?: boolean

4
services/web/frontend/stories/notification.stories.tsx
View File

@@ -300,6 +300,10 @@ export const SuccessFlow = (args: Args) => {
return startNotification
}
export const ContentAsAString = (args: Args) => {
return <Notification {...args} content="An alert" />
}
export default {
title: 'Shared / Components / Notification',
component: Notification,

10
services/web/test/frontend/shared/components/notification.test.tsx
View File

@@ -41,7 +41,7 @@ describe('<Notification />', function () {
screen.getByRole('button', { name: 'Close' })
})
it('renders with title', function () {
it('renders with title and content passed as HTML', function () {
render(
<Notification
type="info"
@@ -50,5 +50,13 @@ describe('<Notification />', function () {
/>
)
screen.getByText('A title')
screen.getByText('A notification')
})
it('renders with content when passed as a string', function () {
render(
<Notification type="info" content="A notification" title="A title" />
)
screen.getByText('A notification')
})
})