Skip to content

Commit d1564f0

Browse files
committed
fix: split multiple status messages onto separate lines
1 parent dd0fbec commit d1564f0

File tree

2 files changed

+17
-7
lines changed

2 files changed

+17
-7
lines changed

packages/webui/src/client/ui/FloatingInspectors/VTFloatingInspector.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ function renderNotice(
3838
noticeLevel: NoticeLevel,
3939
noticeMessages: ReadonlyDeep<ITranslatableMessage[]> | null
4040
): JSX.Element {
41-
const messagesStr = noticeMessages ? noticeMessages.map((msg) => translateMessage(msg, t)).join('; ') : ''
4241
return (
4342
<>
4443
<div className="segment-timeline__mini-inspector__notice-header">
@@ -48,7 +47,14 @@ function renderNotice(
4847
<WarningIconSmall />
4948
) : null}
5049
</div>
51-
<div className="segment-timeline__mini-inspector__notice">{messagesStr}</div>
50+
<div className="segment-timeline__mini-inspector__notice">
51+
{noticeMessages?.map((msg) => (
52+
<>
53+
{translateMessage(msg, t)}
54+
<br />
55+
</>
56+
))}
57+
</div>
5258
</>
5359
)
5460
}

packages/webui/src/client/ui/RundownView/RundownNotifier.tsx

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -603,10 +603,6 @@ class RundownViewNotifier extends WithManagedTracker {
603603

604604
let newNotification: Notification | undefined = undefined
605605
if (status !== PieceStatusCode.OK && status !== PieceStatusCode.UNKNOWN) {
606-
const messagesStr = messages.length
607-
? messages.map((msg) => translateMessage(msg, t)).join('; ')
608-
: t('There is an unspecified problem with the source.')
609-
610606
const issueName = typeof issue.name === 'string' ? issue.name : translateMessage(issue.name, t)
611607
let messageName = issue.segmentName || issueName
612608
if (issue.segmentName && issueName) {
@@ -619,7 +615,15 @@ class RundownViewNotifier extends WithManagedTracker {
619615
(
620616
<>
621617
<h5>{messageName}</h5>
622-
<div>{messagesStr}</div>
618+
<div>
619+
{messages.map((msg) => (
620+
<>
621+
{translateMessage(msg, t)}
622+
<br />
623+
</>
624+
))}
625+
{messages.length === 0 && t('There is an unspecified problem with the source.')}
626+
</div>
623627
</>
624628
),
625629
issue.segmentId ? issue.segmentId : 'line_' + issue.partId,

0 commit comments

Comments
 (0)