From 2309720c0059eba949d904a0e624d3cb1d1ec8d5 Mon Sep 17 00:00:00 2001 From: Quinn Slack Date: Sat, 25 Mar 2023 12:57:42 -0700 Subject: [PATCH] change primary dotcom CTAs to point to App (#49992) The global navbar, `/search`, and `/sign-in` now all point to App as the primary call-to-action instead of Sourcegraph Enterprise. --- client/web/src/auth/SignInPage.tsx | 11 +++++++++-- client/web/src/nav/GlobalNavbar.tsx | 18 ++++++++++-------- .../pages/SearchPage/SearchPageContent.tsx | 16 ++++++++-------- 3 files changed, 27 insertions(+), 18 deletions(-) diff --git a/client/web/src/auth/SignInPage.tsx b/client/web/src/auth/SignInPage.tsx index fd6fe44201a..2436ce630a4 100644 --- a/client/web/src/auth/SignInPage.tsx +++ b/client/web/src/auth/SignInPage.tsx @@ -140,9 +140,16 @@ export const SignInPage: React.FunctionComponentSign up.{' '} {isSourcegraphDotCom && ( <> - To use Sourcegraph on private repositories, + To use Sourcegraph on private repositories,{' '} eventLogger.log('ClickedOnAppCTA', { location: 'SignInPage' })} + > + download Sourcegraph app + {' '} + or{' '} + eventLogger.log('ClickedOnEnterpriseCTA', { location: 'SignInPage' }) } diff --git a/client/web/src/nav/GlobalNavbar.tsx b/client/web/src/nav/GlobalNavbar.tsx index e308cfd951b..5f8056eb3b4 100644 --- a/client/web/src/nav/GlobalNavbar.tsx +++ b/client/web/src/nav/GlobalNavbar.tsx @@ -16,7 +16,7 @@ import { SettingsCascadeProps } from '@sourcegraph/shared/src/settings/settings' import { TelemetryProps } from '@sourcegraph/shared/src/telemetry/telemetryService' import { useIsLightTheme } from '@sourcegraph/shared/src/theme' import { addSourcegraphAppOutboundUrlParameters } from '@sourcegraph/shared/src/util/url' -import { Button, Link, ButtonLink, useWindowSize } from '@sourcegraph/wildcard' +import { Button, Link, ButtonLink, useWindowSize, Tooltip } from '@sourcegraph/wildcard' import { AuthenticatedUser } from '../auth' import { BatchChangesProps } from '../batches' @@ -312,13 +312,15 @@ export const GlobalNavbar: React.FunctionComponent - eventLogger.log('ClickedOnEnterpriseCTA', { location: 'NavBar' })} - > - {props.authenticatedUser && 'Get '} Enterprise - + + eventLogger.log('ClickedOnAppCTA', { location: 'NavBar' })} + > + Download app + + )} {fuzzyFinderNavbar && FuzzyFinderNavItem(props.setFuzzyFinderIsVisible)} diff --git a/client/web/src/storm/pages/SearchPage/SearchPageContent.tsx b/client/web/src/storm/pages/SearchPage/SearchPageContent.tsx index 86719a97a34..ea43c9481c1 100644 --- a/client/web/src/storm/pages/SearchPage/SearchPageContent.tsx +++ b/client/web/src/storm/pages/SearchPage/SearchPageContent.tsx @@ -69,14 +69,14 @@ export const SearchPageContent: FC = props => { Searching millions of public repositories
- - telemetryService.log('ClickedOnEnterpriseCTA', { location: 'HomeAboveSearch' }) - } - > - Get Sourcegraph Enterprise - + + telemetryService.log('ClickedOnAppCTA', { location: 'HomeAboveSearch' })} + > + Download Sourcegraph app + +
)}