✨ log grapher views to google analytics

owid · Oct 29, 2024 · 0a1cf71 · 0a1cf71
1 parent e011027
commit 0a1cf71
Show file tree

Hide file tree

Showing 7 changed files with 106 additions and 71 deletions.
diff --git a/explorer/Explorer.tsx b/explorer/Explorer.tsx
@@ -471,7 +471,6 @@ export class Explorer
         if (mapTargetTime) {
             grapher.map.time = mapTargetTime
         }
-        grapher.slug = this.explorerProgram.slug
         if (!grapher.id) grapher.id = 0
     }
 

diff --git a/packages/@ourworldindata/grapher/src/controls/ActionButtons.tsx b/packages/@ourworldindata/grapher/src/controls/ActionButtons.tsx
@@ -203,7 +203,9 @@ export class ActionButtons extends React.Component<{
     }
 
     @computed private get hasShareButton(): boolean {
-        return !this.manager.hideShareButton
+        return (
+            !this.manager.hideShareButton && ShareMenu.shouldShow(this.manager)
+        )
     }
 
     @computed private get hasFullScreenButton(): boolean {

diff --git a/packages/@ourworldindata/grapher/src/controls/ShareMenu.scss b/packages/@ourworldindata/grapher/src/controls/ShareMenu.scss
@@ -56,8 +56,3 @@ $zindex-ControlsFooter: 2;
         }
     }
 }
-
-.ShareMenu.disabled a {
-    pointer-events: none;
-    opacity: 0.3;
-}
diff --git a/packages/@ourworldindata/grapher/src/controls/ShareMenu.tsx b/packages/@ourworldindata/grapher/src/controls/ShareMenu.tsx
@@ -21,7 +21,7 @@ export interface ShareMenuManager {
 
 interface ShareMenuProps {
     manager: ShareMenuManager
-    onDismiss: () => void
+    onDismiss?: () => void
     right?: number
 }
 
@@ -91,6 +91,11 @@ export class ShareMenu extends React.Component<ShareMenuProps, ShareMenuState> {
         }
     }
 
+    static shouldShow(manager: ShareMenuManager): boolean {
+        const test = new ShareMenu({ manager })
+        return test.showShareMenu
+    }
+
     @computed get manager(): ShareMenuManager {
         return this.props.manager
     }
@@ -99,16 +104,16 @@ export class ShareMenu extends React.Component<ShareMenuProps, ShareMenuState> {
         return this.manager.currentTitle ?? ""
     }
 
-    @computed get isDisabled(): boolean {
-        return !this.manager.slug
+    @computed get showShareMenu(): boolean {
+        return !!this.canonicalUrl || !!this.manager.editUrl
     }
 
     @computed get canonicalUrl(): string | undefined {
         return this.manager.canonicalUrl
     }
 
     @action.bound dismiss(): void {
-        this.props.onDismiss()
+        this.props.onDismiss?.()
     }
 
     @action.bound onClickSomewhere(): void {
@@ -153,35 +158,31 @@ export class ShareMenu extends React.Component<ShareMenuProps, ShareMenuState> {
         }
     }
 
-    @computed get twitterHref(): string {
-        let href =
-            "https://twitter.com/intent/tweet/?text=" +
-            encodeURIComponent(this.title)
-        if (this.canonicalUrl)
-            href += "&url=" + encodeURIComponent(this.canonicalUrl)
-        return href
+    @computed get twitterHref(): string | undefined {
+        if (!this.canonicalUrl) return undefined
+        const queryParams = new URLSearchParams({
+            text: this.title,
+            url: this.canonicalUrl,
+        })
+        return `https://twitter.com/intent/tweet/?${queryParams}`
     }
 
-    @computed get facebookHref(): string {
-        let href =
-            "https://www.facebook.com/dialog/share?app_id=1149943818390250&display=page"
-        if (this.canonicalUrl)
-            href += "&href=" + encodeURIComponent(this.canonicalUrl)
-        return href
+    @computed get facebookHref(): string | undefined {
+        if (!this.canonicalUrl) return undefined
+        const queryParams = new URLSearchParams({
+            app_id: "1149943818390250",
+            display: "page",
+            href: this.canonicalUrl,
+        })
+        return `https://www.facebook.com/dialog/share?${queryParams}`
     }
 
     @computed get canUseShareApi(): boolean {
         return canUseShareApi(this.manager)
     }
 
     render(): React.ReactElement {
-        const {
-            twitterHref,
-            facebookHref,
-            isDisabled,
-            canUseShareApi,
-            manager,
-        } = this
+        const { twitterHref, facebookHref, canUseShareApi, manager } = this
         const { editUrl } = manager
 
         const width = 200
@@ -193,46 +194,52 @@ export class ShareMenu extends React.Component<ShareMenuProps, ShareMenuState> {
 
         return (
             <div
-                className={"ShareMenu" + (isDisabled ? " disabled" : "")}
+                className="ShareMenu"
                 onClick={action(() => (this.dismissable = false))}
                 style={style}
             >
                 <h2>Share</h2>
-                <a
-                    target="_blank"
-                    title="Tweet a link"
-                    data-track-note="chart_share_twitter"
-                    href={twitterHref}
-                    rel="noopener"
-                >
-                    <span className="icon">
-                        <FontAwesomeIcon icon={faXTwitter} />
-                    </span>{" "}
-                    X/Twitter
-                </a>
-                <a
-                    target="_blank"
-                    title="Share on Facebook"
-                    data-track-note="chart_share_facebook"
-                    href={facebookHref}
-                    rel="noopener"
-                >
-                    <span className="icon">
-                        <FontAwesomeIcon icon={faFacebook} />
-                    </span>{" "}
-                    Facebook
-                </a>
-                <a
-                    className="embed"
-                    title="Embed this visualization in another HTML document"
-                    data-track-note="chart_share_embed"
-                    onClick={this.onEmbed}
-                >
-                    <span className="icon">
-                        <FontAwesomeIcon icon={faCode} />
-                    </span>{" "}
-                    Embed
-                </a>
+                {twitterHref && (
+                    <a
+                        target="_blank"
+                        title="Tweet a link"
+                        data-track-note="chart_share_twitter"
+                        href={twitterHref}
+                        rel="noopener"
+                    >
+                        <span className="icon">
+                            <FontAwesomeIcon icon={faXTwitter} />
+                        </span>{" "}
+                        X/Twitter
+                    </a>
+                )}
+                {facebookHref && (
+                    <a
+                        target="_blank"
+                        title="Share on Facebook"
+                        data-track-note="chart_share_facebook"
+                        href={facebookHref}
+                        rel="noopener"
+                    >
+                        <span className="icon">
+                            <FontAwesomeIcon icon={faFacebook} />
+                        </span>{" "}
+                        Facebook
+                    </a>
+                )}
+                {this.canonicalUrl && (
+                    <a
+                        className="embed"
+                        title="Embed this visualization in another HTML document"
+                        data-track-note="chart_share_embed"
+                        onClick={this.onEmbed}
+                    >
+                        <span className="icon">
+                            <FontAwesomeIcon icon={faCode} />
+                        </span>{" "}
+                        Embed
+                    </a>
+                )}
                 {canUseShareApi && (
                     <a
                         title="Share this visualization with an app on your device"
@@ -245,7 +252,7 @@ export class ShareMenu extends React.Component<ShareMenuProps, ShareMenuState> {
                         Share via&hellip;
                     </a>
                 )}
-                {this.state.canWriteToClipboard && (
+                {this.state.canWriteToClipboard && this.canonicalUrl && (
                     <a
                         title="Copy link to clipboard"
                         data-track-note="chart_share_copylink"

diff --git a/packages/@ourworldindata/grapher/src/core/Grapher.tsx b/packages/@ourworldindata/grapher/src/core/Grapher.tsx
@@ -943,7 +943,8 @@ export class Grapher
             devtools: {
                 track: "Grapher",
                 properties: [
-                    ["slug", this.slug],
+                    // might be missing for charts within explorers or mdims
+                    ["slug", this.slug ?? "missing-slug"],
                     ["chartType", this.type],
                     ["tab", this.tab],
                 ],
@@ -2342,6 +2343,7 @@ export class Grapher
         return this.base.current || undefined
     }
 
+    private hasLoggedGAViewEvent = false
     @observable private hasBeenVisible = false
     @observable private uncaughtError?: Error
 
@@ -2829,6 +2831,12 @@ export class Grapher
                 entries.forEach((entry) => {
                     if (entry.isIntersecting) {
                         this.hasBeenVisible = true
+
+                        if (this.slug && !this.hasLoggedGAViewEvent) {
+                            this.hasLoggedGAViewEvent = true
+                            this.analytics.logGrapherView(this.slug)
+                        }
+
                         observer.disconnect()
                     }
                 })

diff --git a/packages/@ourworldindata/grapher/src/core/GrapherAnalytics.ts b/packages/@ourworldindata/grapher/src/core/GrapherAnalytics.ts
@@ -12,6 +12,7 @@ export enum EventCategory {
     CountryProfileSearch = "owid.country_profile_search",
     Filter = "owid.filter",
     GlobalEntitySelectorUsage = "owid.global_entity_selector_usage",
+    GrapherView = "owid.grapher_view",
     GrapherClick = "owid.grapher_click",
     GrapherError = "owid.grapher_error",
     ExplorerCountrySelector = "owid.explorer_country_selector",
@@ -48,6 +49,7 @@ interface GAEvent {
     eventContext?: string
     eventTarget?: string
     grapherPath?: string
+    grapherView?: string
 }
 
 // taken from https://github.com/DefinitelyTyped/DefinitelyTyped/blob/de66435d18fbdb2684947d16b5cd3a77f876324c/types/gtag.js/index.d.ts#L151-L156
@@ -71,6 +73,14 @@ export class GrapherAnalytics {
     private version: string // Ideally the Git hash commit
     private isDev: boolean
 
+    logGrapherView(slug: string, view?: string): void {
+        this.logToGA({
+            event: EventCategory.GrapherView,
+            grapherPath: `/grapher/${slug}`,
+            grapherView: view,
+        })
+    }
+
     logGrapherViewError(error: Error): void {
         this.logToGA({
             event: EventCategory.GrapherError,
@@ -184,6 +194,9 @@ export class GrapherAnalytics {
     }
 
     protected logToGA(event: GAEvent): void {
+        // TODO: remove!! added for testing on staging
+        console.log("Analytics.logToGA", event)
+
         if (DEBUG && this.isDev) {
             // eslint-disable-next-line no-console
             console.log("Analytics.logToGA", event)

diff --git a/site/multiDim/MultiDimDataPageContent.tsx b/site/multiDim/MultiDimDataPageContent.tsx
@@ -1,6 +1,7 @@
 import React, { useCallback, useEffect, useMemo, useRef, useState } from "react"
 import {
     Grapher,
+    GrapherAnalytics,
     GrapherProgrammaticInterface,
     getVariableMetadataRoute,
 } from "@ourworldindata/grapher"
@@ -201,6 +202,8 @@ const useVarDatapageData = (
     }
 }
 
+const analytics = new GrapherAnalytics()
+
 export const MultiDimDataPageContent = ({
     // _datapageData,
     configObj,
@@ -215,6 +218,8 @@ export const MultiDimDataPageContent = ({
 }: MultiDimDataPageProps) => {
     const grapherFigureRef = useRef<HTMLDivElement>(null)
 
+    const slug = window?.location.pathname.split("/").pop()
+
     const config = useMemo(
         () => MultiDimDataPageConfig.fromObject(configObj),
         [configObj]
@@ -270,6 +275,13 @@ export const MultiDimDataPageContent = ({
         setWindowQueryStr(queryStr ?? "")
     }, [queryStr])
 
+    useEffect(() => {
+        if (slug) {
+            const view = new URLSearchParams(currentSettings).toString()
+            analytics.logGrapherView(slug, view)
+        }
+    }, [slug, currentSettings])
+
     const grapherConfigComputed = useMemo(() => {
         const baseConfig: GrapherProgrammaticInterface = {
             selectedEntityNames: config.config.defaultSelection ?? [],
@@ -284,12 +296,11 @@ export const MultiDimDataPageContent = ({
             dimensions: dimensionsConfig,
             ...baseConfig,
             dataApiUrl: DATA_API_URL,
-            // TODO: The way manager and slug are set here are just workarounds to make the edit button in the
+            // TODO: The way manager is set here is just a workaround to make the edit button in the
             // share menu work. They should be removed before we publish MDims!
             manager: {
                 canonicalUrl,
             },
-            slug: "DUMMY",
         } as GrapherProgrammaticInterface
     }, [
         varGrapherConfig,