fix: consolidate browser notification learn more text into single localized string (#4716)

- Combines 'Learn more' link and surrounding text into one Translation
- Fixes double space issue (was   + regular space)
- Enables full sentence to be localized: '<a>Learn more</a> about Owncast browser notifications.'

Closes #4655
This commit is contained in:
John Costa
2026-01-14 12:57:14 -08:00
committed by GitHub
parent da847bac78
commit 9c747e865a
2 changed files with 6 additions and 10 deletions
@@ -282,15 +282,10 @@ export const BrowserNotifyModal = () => {
translationKey={Localization.Frontend.BrowserNotifyModal.mainDescription}
defaultText="Get notified right in the browser each time this stream goes live."
/>
<span>
<a href="https://owncast.online/docs/notifications/#browser-notifications">
<Translation
translationKey={Localization.Frontend.BrowserNotifyModal.learnMore}
defaultText="Learn more"
/>
</a>
&nbsp; about Owncast browser notifications.
</span>
<Translation
translationKey={Localization.Frontend.BrowserNotifyModal.learnMoreAboutNotifications}
defaultText="<a href='https://owncast.online/docs/notifications/#browser-notifications'>Learn more</a> about Owncast browser notifications."
/>
</Row>
<Row>
{error && (