From d9443794f67990e3920348ef41c042ad1fa963e2 Mon Sep 17 00:00:00 2001 From: Stephen Donner Date: Sat, 24 Jul 2021 12:10:06 -0700 Subject: [PATCH 1/3] Change Brave Talk link to talk.brave.com/widget Fixes https://github.com/brave/brave-browser/issues/17146 --- .../brave_new_tab_ui/components/default/together/index.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/brave_new_tab_ui/components/default/together/index.tsx b/components/brave_new_tab_ui/components/default/together/index.tsx index b71c8c23a698..a513d5b7e401 100644 --- a/components/brave_new_tab_ui/components/default/together/index.tsx +++ b/components/brave_new_tab_ui/components/default/together/index.tsx @@ -60,7 +60,7 @@ class Together extends React.PureComponent { shouldCreateCall = (event: any) => { event.preventDefault() - window.open(`https://together.brave.com/widget`, '_self', 'noopener') + window.open(`https://talk.brave.com/widget`, '_self', 'noopener') } render () { From 58436fc530214d5dd5abf3c28a0e22084e647677 Mon Sep 17 00:00:00 2001 From: Stephen Donner Date: Sat, 24 Jul 2021 12:30:16 -0700 Subject: [PATCH 2/3] Update Try it out button link to talk.brave.com/widget Fixes https://github.com/brave/brave-browser/issues/17147 --- .../components/default/footer/togetherTooltip/index.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx b/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx index f2a9ca9729df..0fac798a7839 100644 --- a/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx +++ b/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx @@ -25,7 +25,7 @@ const TogetherTooltip: React.FunctionComponent = function (props) { {getLocale('togetherPromptDescription')} - + {getLocale('togetherPromptAction')} Date: Wed, 28 Jul 2021 11:35:24 -0700 Subject: [PATCH 3/3] Update Brave Talk to use constant for Widget URL --- .../components/default/footer/togetherTooltip/index.tsx | 3 ++- .../brave_new_tab_ui/components/default/together/index.tsx | 3 ++- components/brave_new_tab_ui/constants/new_tab_ui.ts | 1 + 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx b/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx index 0fac798a7839..812a5435f228 100644 --- a/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx +++ b/components/brave_new_tab_ui/components/default/footer/togetherTooltip/index.tsx @@ -9,6 +9,7 @@ import TogetherIcon from './togetherIcon' import CloseIcon from './closeIcon' import { LinkButton } from '../../../outlineButton' import * as S from './style' +import { braveTalkWidgetUrl } from '../../../../constants/new_tab_ui' type Props = { onClose: () => any @@ -25,7 +26,7 @@ const TogetherTooltip: React.FunctionComponent = function (props) { {getLocale('togetherPromptDescription')} - + {getLocale('togetherPromptAction')} { shouldCreateCall = (event: any) => { event.preventDefault() - window.open(`https://talk.brave.com/widget`, '_self', 'noopener') + window.open(braveTalkWidgetUrl, '_self', 'noopener') } render () { diff --git a/components/brave_new_tab_ui/constants/new_tab_ui.ts b/components/brave_new_tab_ui/constants/new_tab_ui.ts index 43156832adcf..cd3f9c9a3a01 100644 --- a/components/brave_new_tab_ui/constants/new_tab_ui.ts +++ b/components/brave_new_tab_ui/constants/new_tab_ui.ts @@ -6,3 +6,4 @@ // Sync this with kBraveMaxTopSitesNumber // in //brave/browser/ui/webui/new_tab_page/brave_new_tab_ui_unittest.cc export const MAX_GRID_SIZE = 12 +export const braveTalkWidgetUrl = 'https://talk.brave.com/widget'