From b65ff1ffdb23e51a4c45e5b5b673f11b5f2525c9 Mon Sep 17 00:00:00 2001
From: MinamiFunakoshiTR <minami.funakoshi@thomsonreuters.com>
Date: Tue, 11 Mar 2025 13:34:43 -0700
Subject: [PATCH 1/2] cleans up old .md files

---
 src/components/Analytics/Analytics.mdx        |  4 +--
 src/components/Analytics/Analytics.svelte     |  3 +-
 .../Analytics/stories/docs/component.md       | 11 -------
 .../Analytics/stories/docs/environments.md    | 17 ----------
 .../Analytics/stories/docs/multipage.md       | 31 -------------------
 5 files changed, 4 insertions(+), 62 deletions(-)
 delete mode 100644 src/components/Analytics/stories/docs/component.md
 delete mode 100644 src/components/Analytics/stories/docs/environments.md
 delete mode 100644 src/components/Analytics/stories/docs/multipage.md

diff --git a/src/components/Analytics/Analytics.mdx b/src/components/Analytics/Analytics.mdx
index 8b39f502..60e7afd2 100644
--- a/src/components/Analytics/Analytics.mdx
+++ b/src/components/Analytics/Analytics.mdx
@@ -6,7 +6,7 @@ import * as AnalyticsStories from './Analytics.stories.svelte';
 
 # Analytics
 
-Add Google and Chartbeat analytics to your page.
+The `Analytics` component adds Google and Chartbeat analytics to your page.
 
 ```svelte
 <script>
@@ -42,7 +42,7 @@ For example, the following excludes analytics from pages in development or hoste
 
 If you're using analytics to measure a multipage newsapp that uses [client-side routing](https://kit.svelte.dev/docs/glossary#routing), then you may need to trigger analytics after virtual page navigation.
 
-This component also exports a function you can call to register pageviews.
+This component exports a function you can call to register pageviews.
 
 For example, here's how you can use SvelteKit's [`afterNavigate`](https://kit.svelte.dev/docs/modules#$app-navigation-afternavigate) lifecycle to capture additional pageviews:
 
diff --git a/src/components/Analytics/Analytics.svelte b/src/components/Analytics/Analytics.svelte
index c623e547..e8b5ce14 100644
--- a/src/components/Analytics/Analytics.svelte
+++ b/src/components/Analytics/Analytics.svelte
@@ -1,3 +1,5 @@
+<!-- @component `Analytics` [Read the docs.](https://reuters-graphics.github.io/graphics-components/?path=/docs/components-ads-analytics-analytics--docs) -->
+
 <script module>
   import { registerPageview as registerChartbeatPageview } from './providers/chartbeat';
   import { registerPageview as registerGAPageview } from './providers/ga';
@@ -13,7 +15,6 @@
   export { registerPageview };
 </script>
 
-<!-- @component `Analytics` [Read the docs.](https://reuters-graphics.github.io/graphics-components/?path=/docs/components-ads-analytics-analytics--docs) -->
 <script lang="ts">
   interface Author {
     name: string;
diff --git a/src/components/Analytics/stories/docs/component.md b/src/components/Analytics/stories/docs/component.md
deleted file mode 100644
index 1ddd7b8e..00000000
--- a/src/components/Analytics/stories/docs/component.md
+++ /dev/null
@@ -1,11 +0,0 @@
-Add Google and Chartbeat analytics to your page.
-
-```svelte
-<script>
-  import { Analytics } from '@reuters-graphics/graphics-components';
-
-  const authors = [{ name: 'Jane Doe' }, { name: 'John Doe' }];
-</script>
-
-<Analytics authors="{authors}" />
-```
diff --git a/src/components/Analytics/stories/docs/environments.md b/src/components/Analytics/stories/docs/environments.md
deleted file mode 100644
index 74089219..00000000
--- a/src/components/Analytics/stories/docs/environments.md
+++ /dev/null
@@ -1,17 +0,0 @@
-Generally, you only want to send page analytics in production environments.
-
-In a SvelteKit context, you can use `$app` stores to restrict when you send analytics.
-
-For example, the following excludes analytics from pages in development or hosted on our preview server:
-
-```svelte
-<script>
-  import { Analytics } from '@reuters-graphics/graphics-components';
-  import { dev } from '$app/environment';
-  import { page } from '$app/stores';
-</script>
-
-{#if !dev && $page.url?.hostname !== 'graphics.thomsonreuters.com'}
-  <Analytics />
-{/if}
-```
diff --git a/src/components/Analytics/stories/docs/multipage.md b/src/components/Analytics/stories/docs/multipage.md
deleted file mode 100644
index a3223927..00000000
--- a/src/components/Analytics/stories/docs/multipage.md
+++ /dev/null
@@ -1,31 +0,0 @@
-If you're using analytics to measure a multipage newsapp that uses [client-side routing](https://kit.svelte.dev/docs/glossary#routing), then you may need to trigger analytics after virtual page navigation.
-
-This component also exports a function you can call to register pageviews.
-
-For example, here's how you can use SvelteKit's [`afterNavigate`](https://kit.svelte.dev/docs/modules#$app-navigation-afternavigate) lifecycle to capture additional pageviews:
-
-```svelte
-<script>
-  import {
-    Analytics,
-    registerPageview,
-  } from '@reuters-graphics/graphics-components';
-  import { afterNavigate } from '$app/navigation';
-
-  let isFirstPageview = true;
-
-  afterNavigate(() => {
-    // We shouldn't fire on initial page load because the Analytics component
-    // already registers a reader's first pageview. After this component
-    // has initially mounted, we can be sure that further navigation is virtual
-    // and register pageviews using this function.
-    if (!isFirstPageview) {
-      registerPageview();
-    } else {
-      isFirstPageview = false;
-    }
-  });
-</script>
-
-<Analytics />
-```

From 08e088e707f444814b535818abf17d1bd310fb38 Mon Sep 17 00:00:00 2001
From: MinamiFunakoshiTR <minami.funakoshi@thomsonreuters.com>
Date: Thu, 13 Mar 2025 13:57:49 -0700
Subject: [PATCH 2/2] removes demo

---
 src/components/Analytics/Analytics.stories.svelte | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/components/Analytics/Analytics.stories.svelte b/src/components/Analytics/Analytics.stories.svelte
index bbb7031f..cf9c24a1 100644
--- a/src/components/Analytics/Analytics.stories.svelte
+++ b/src/components/Analytics/Analytics.stories.svelte
@@ -1,5 +1,5 @@
 <script module lang="ts">
-  import { defineMeta, type Args } from '@storybook/addon-svelte-csf';
+  import { defineMeta } from '@storybook/addon-svelte-csf';
   import Analytics from './Analytics.svelte';
 
   const { Story } = defineMeta({
@@ -8,15 +8,10 @@
   });
 </script>
 
-{#snippet template(args: Args<typeof Story>)}
-  <Analytics {...args} />
-  <div>Nothing to see here</div>
-{/snippet}
-
 <Story
   name="Demo"
+  tags={['!autodocs', '!dev']}
   args={{
     authors: [{ name: 'Jane Doe' }, { name: 'John Doe' }],
   }}
-  children={template}
 />