From 4baf32b2bf749a7bcc7ec995d77590e76a7439ee Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Tue, 5 Sep 2023 15:34:38 +0200
Subject: [PATCH 1/9] fix: show warning text below RichTextEditor when unable
 to show snapshot

---
 src/__demo__/RichTextEditor.stories.js        | 28 +++++++++++++++++++
 .../InterpretationsUnit/InterpretationForm.js |  9 ++++++
 .../InterpretationsUnit.js                    |  7 +++++
 .../common/RichTextEditor/RichTextEditor.js   | 10 +++++--
 4 files changed, 52 insertions(+), 2 deletions(-)
 create mode 100644 src/__demo__/RichTextEditor.stories.js

diff --git a/src/__demo__/RichTextEditor.stories.js b/src/__demo__/RichTextEditor.stories.js
new file mode 100644
index 000000000..e4b669cd7
--- /dev/null
+++ b/src/__demo__/RichTextEditor.stories.js
@@ -0,0 +1,28 @@
+import { CustomDataProvider } from '@dhis2/app-runtime'
+import { storiesOf } from '@storybook/react'
+import React, { useState } from 'react'
+import { RichTextEditor } from '../components/Interpretations/common/index.js'
+
+storiesOf('RichTextEditor', module).add('Default', () => {
+    const [value, setValue] = useState('')
+
+    return (
+        <CustomDataProvider>
+            <RichTextEditor value={value} onChange={setValue} />
+        </CustomDataProvider>
+    )
+})
+
+storiesOf('RichTextEditor', module).add('Long warning text', () => {
+    const [value, setValue] = useState('')
+
+    return (
+        <CustomDataProvider>
+            <RichTextEditor
+                value={value}
+                onChange={setValue}
+                warningText="Unable to create a snapshot of this visualization, because it contains no time dimension. Beware that others viewing this interpretation in the future may see more data than currently available."
+            />
+        </CustomDataProvider>
+    )
+})
diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
index 1fa8b471d..9babb1ec2 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
@@ -14,6 +14,7 @@ export const InterpretationForm = ({
     id,
     currentUser,
     disabled,
+    showNoTimeDimensionWarning,
     onSave,
 }) => {
     const [showRichTextEditor, setShowRichTextEditor] = useState(false)
@@ -51,6 +52,13 @@ export const InterpretationForm = ({
                         inputPlaceholder={inputPlaceholder}
                         onChange={setInterpretationText}
                         value={interpretationText}
+                        warningText={
+                            showNoTimeDimensionWarning
+                                ? i18n.t(
+                                      'Unable to create a snapshot of this visualization, because it contains no time dimension. Beware that others viewing this interpretation in the future may see more data than currently available.'
+                                  )
+                                : undefined
+                        }
                     />
                     <MessageButtonStrip>
                         <Button
@@ -89,6 +97,7 @@ InterpretationForm.propTypes = {
     currentUser: PropTypes.object,
     disabled: PropTypes.bool,
     id: PropTypes.string,
+    showNoTimeDimensionWarning: PropTypes.bool,
     type: PropTypes.string,
     onSave: PropTypes.func,
 }
diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
index 1a3ce45fd..e3377c638 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
@@ -44,6 +44,7 @@ export const InterpretationsUnit = forwardRef(
             currentUser,
             type,
             id,
+            visualizationHasTimeDimension,
             onInterpretationClick,
             onReplyIconClick,
             disabled,
@@ -52,6 +53,8 @@ export const InterpretationsUnit = forwardRef(
         ref
     ) => {
         const [isExpanded, setIsExpanded] = useState(true)
+        const showNoTimeDimensionWarning =
+            type === 'eventVisualization' && !visualizationHasTimeDimension
 
         const { data, loading, fetching, refetch } = useDataQuery(
             interpretationsQuery,
@@ -115,6 +118,9 @@ export const InterpretationsUnit = forwardRef(
                                     id={id}
                                     onSave={onCompleteAction}
                                     disabled={disabled}
+                                    showNoTimeDimensionWarning={
+                                        showNoTimeDimensionWarning
+                                    }
                                 />
                                 <InterpretationList
                                     currentUser={currentUser}
@@ -189,6 +195,7 @@ InterpretationsUnit.propTypes = {
     type: PropTypes.string.isRequired,
     disabled: PropTypes.bool,
     renderId: PropTypes.number,
+    visualizationHasTimeDimension: PropTypes.bool,
     onInterpretationClick: PropTypes.func,
     onReplyIconClick: PropTypes.func,
 }
diff --git a/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js b/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
index ba2eba9fe..853f5ffde 100644
--- a/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
+++ b/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
@@ -192,12 +192,13 @@ Toolbar.propTypes = {
 
 export const RichTextEditor = forwardRef(
     (
-        { value, disabled, inputPlaceholder, onChange, errorText },
+        { value, disabled, inputPlaceholder, onChange, errorText, warningText },
         externalRef
     ) => {
         const [previewMode, setPreviewMode] = useState(false)
         const internalRef = useRef()
         const textareaRef = externalRef || internalRef
+        const validationText = errorText ?? warningText
 
         useEffect(() => textareaRef.current?.focus(), [textareaRef])
 
@@ -234,7 +235,11 @@ export const RichTextEditor = forwardRef(
                         <RichTextParser>{value}</RichTextParser>
                     </div>
                 ) : (
-                    <Field error={!!errorText} validationText={errorText}>
+                    <Field
+                        error={!!errorText}
+                        warning={!errorText && !!warningText}
+                        validationText={validationText}
+                    >
                         <UserMentionWrapper
                             onUserSelect={onChange}
                             inputReference={textareaRef}
@@ -269,4 +274,5 @@ RichTextEditor.propTypes = {
     disabled: PropTypes.bool,
     errorText: PropTypes.string,
     inputPlaceholder: PropTypes.string,
+    warningText: PropTypes.string,
 }

From 020168a4a11d73ce4b460d2c61d29afe88191068 Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Tue, 5 Sep 2023 15:42:47 +0200
Subject: [PATCH 2/9] chore: update pot file

---
 i18n/en.pot | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/i18n/en.pot b/i18n/en.pot
index 52645c02d..a7fca475c 100644
--- a/i18n/en.pot
+++ b/i18n/en.pot
@@ -5,8 +5,8 @@ msgstr ""
 "Content-Type: text/plain; charset=utf-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=(n != 1)\n"
-"POT-Creation-Date: 2023-07-06T08:30:33.216Z\n"
-"PO-Revision-Date: 2023-07-06T08:30:33.216Z\n"
+"POT-Creation-Date: 2023-09-05T13:40:21.351Z\n"
+"PO-Revision-Date: 2023-09-05T13:40:21.351Z\n"
 
 msgid "view only"
 msgstr "view only"
@@ -374,6 +374,15 @@ msgstr "Hide interpretation"
 msgid "Write an interpretation"
 msgstr "Write an interpretation"
 
+msgid ""
+"Unable to create a snapshot of this visualization, because it contains no "
+"time dimension. Beware that others viewing this interpretation in the "
+"future may see more data than currently available."
+msgstr ""
+"Unable to create a snapshot of this visualization, because it contains no "
+"time dimension. Beware that others viewing this interpretation in the "
+"future may see more data than currently available."
+
 msgid "Post interpretation"
 msgstr "Post interpretation"
 

From b8ed803ffdeccddae2be74d2c1f08cd569c1e320 Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Tue, 5 Sep 2023 16:37:04 +0200
Subject: [PATCH 3/9] chore: replace rich-text-editor stories by
 interpretations-unit stories

---
 src/__demo__/InterpretationsUnit.stories.js | 51 +++++++++++++++++++++
 src/__demo__/RichTextEditor.stories.js      | 28 -----------
 2 files changed, 51 insertions(+), 28 deletions(-)
 create mode 100644 src/__demo__/InterpretationsUnit.stories.js
 delete mode 100644 src/__demo__/RichTextEditor.stories.js

diff --git a/src/__demo__/InterpretationsUnit.stories.js b/src/__demo__/InterpretationsUnit.stories.js
new file mode 100644
index 000000000..a02ba098c
--- /dev/null
+++ b/src/__demo__/InterpretationsUnit.stories.js
@@ -0,0 +1,51 @@
+import { CustomDataProvider } from '@dhis2/app-runtime'
+import { storiesOf } from '@storybook/react'
+import React from 'react'
+import { InterpretationsUnit } from '../components/Interpretations/InterpretationsUnit/index.js'
+
+storiesOf('IntepretationsUnit', module).add('Default', () => {
+    return (
+        <CustomDataProvider
+            data={{
+                interpretations: {
+                    interpretations: [],
+                },
+            }}
+        >
+            <InterpretationsUnit
+                currentUser={{
+                    name: 'Tom Wakiki',
+                }}
+                id="abcd"
+                type="eventVisualization"
+                onReplyIconClick={Function.prototype}
+                visualizationHasTimeDimension={true}
+            />
+        </CustomDataProvider>
+    )
+})
+
+storiesOf('IntepretationsUnit', module).add(
+    'With no time dimensions warning',
+    () => {
+        return (
+            <CustomDataProvider
+                data={{
+                    interpretations: {
+                        interpretations: [],
+                    },
+                }}
+            >
+                <InterpretationsUnit
+                    currentUser={{
+                        name: 'Tom Wakiki',
+                    }}
+                    id="abcd"
+                    type="eventVisualization"
+                    onReplyIconClick={Function.prototype}
+                    visualizationHasTimeDimension={false}
+                />
+            </CustomDataProvider>
+        )
+    }
+)
diff --git a/src/__demo__/RichTextEditor.stories.js b/src/__demo__/RichTextEditor.stories.js
deleted file mode 100644
index e4b669cd7..000000000
--- a/src/__demo__/RichTextEditor.stories.js
+++ /dev/null
@@ -1,28 +0,0 @@
-import { CustomDataProvider } from '@dhis2/app-runtime'
-import { storiesOf } from '@storybook/react'
-import React, { useState } from 'react'
-import { RichTextEditor } from '../components/Interpretations/common/index.js'
-
-storiesOf('RichTextEditor', module).add('Default', () => {
-    const [value, setValue] = useState('')
-
-    return (
-        <CustomDataProvider>
-            <RichTextEditor value={value} onChange={setValue} />
-        </CustomDataProvider>
-    )
-})
-
-storiesOf('RichTextEditor', module).add('Long warning text', () => {
-    const [value, setValue] = useState('')
-
-    return (
-        <CustomDataProvider>
-            <RichTextEditor
-                value={value}
-                onChange={setValue}
-                warningText="Unable to create a snapshot of this visualization, because it contains no time dimension. Beware that others viewing this interpretation in the future may see more data than currently available."
-            />
-        </CustomDataProvider>
-    )
-})

From 2e0b679dacd0e4d8fac34b592006be091473937c Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Tue, 5 Sep 2023 16:38:15 +0200
Subject: [PATCH 4/9] chore: change props order to make things clearer

---
 src/__demo__/InterpretationsUnit.stories.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/__demo__/InterpretationsUnit.stories.js b/src/__demo__/InterpretationsUnit.stories.js
index a02ba098c..f1c786f57 100644
--- a/src/__demo__/InterpretationsUnit.stories.js
+++ b/src/__demo__/InterpretationsUnit.stories.js
@@ -17,8 +17,8 @@ storiesOf('IntepretationsUnit', module).add('Default', () => {
                     name: 'Tom Wakiki',
                 }}
                 id="abcd"
-                type="eventVisualization"
                 onReplyIconClick={Function.prototype}
+                type="eventVisualization"
                 visualizationHasTimeDimension={true}
             />
         </CustomDataProvider>
@@ -41,8 +41,8 @@ storiesOf('IntepretationsUnit', module).add(
                         name: 'Tom Wakiki',
                     }}
                     id="abcd"
-                    type="eventVisualization"
                     onReplyIconClick={Function.prototype}
+                    type="eventVisualization"
                     visualizationHasTimeDimension={false}
                 />
             </CustomDataProvider>

From 745f153b77fab0c0329335c26a5d91d1ade3d6e5 Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Mon, 11 Sep 2023 15:23:27 +0200
Subject: [PATCH 5/9] fix: use help-text instead of validationText in warning
 state

---
 .../InterpretationsUnit/InterpretationForm.js         | 11 ++++++-----
 .../InterpretationsUnit/InterpretationsUnit.js        |  6 +++---
 .../common/RichTextEditor/RichTextEditor.js           |  9 ++++-----
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
index 9babb1ec2..27cc2a422 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
@@ -14,7 +14,7 @@ export const InterpretationForm = ({
     id,
     currentUser,
     disabled,
-    showNoTimeDimensionWarning,
+    showNoTimeDimensionHelpText,
     onSave,
 }) => {
     const [showRichTextEditor, setShowRichTextEditor] = useState(false)
@@ -40,6 +40,7 @@ export const InterpretationForm = ({
 
     const inputPlaceholder = i18n.t('Write an interpretation')
 
+    console.log(showNoTimeDimensionHelpText)
     return (
         <MessageEditorContainer
             currentUser={currentUser}
@@ -52,10 +53,10 @@ export const InterpretationForm = ({
                         inputPlaceholder={inputPlaceholder}
                         onChange={setInterpretationText}
                         value={interpretationText}
-                        warningText={
-                            showNoTimeDimensionWarning
+                        helpText={
+                            showNoTimeDimensionHelpText
                                 ? i18n.t(
-                                      'Unable to create a snapshot of this visualization, because it contains no time dimension. Beware that others viewing this interpretation in the future may see more data than currently available.'
+                                      'Visualization has no time dimension. Other people viewing interpretations in the future may see more data.'
                                   )
                                 : undefined
                         }
@@ -97,7 +98,7 @@ InterpretationForm.propTypes = {
     currentUser: PropTypes.object,
     disabled: PropTypes.bool,
     id: PropTypes.string,
-    showNoTimeDimensionWarning: PropTypes.bool,
+    showNoTimeDimensionHelpText: PropTypes.bool,
     type: PropTypes.string,
     onSave: PropTypes.func,
 }
diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
index e3377c638..17359ea4f 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
@@ -53,7 +53,7 @@ export const InterpretationsUnit = forwardRef(
         ref
     ) => {
         const [isExpanded, setIsExpanded] = useState(true)
-        const showNoTimeDimensionWarning =
+        const showNoTimeDimensionHelpText =
             type === 'eventVisualization' && !visualizationHasTimeDimension
 
         const { data, loading, fetching, refetch } = useDataQuery(
@@ -118,8 +118,8 @@ export const InterpretationsUnit = forwardRef(
                                     id={id}
                                     onSave={onCompleteAction}
                                     disabled={disabled}
-                                    showNoTimeDimensionWarning={
-                                        showNoTimeDimensionWarning
+                                    showNoTimeDimensionHelpText={
+                                        showNoTimeDimensionHelpText
                                     }
                                 />
                                 <InterpretationList
diff --git a/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js b/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
index 853f5ffde..e8ad9216d 100644
--- a/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
+++ b/src/components/Interpretations/common/RichTextEditor/RichTextEditor.js
@@ -192,13 +192,12 @@ Toolbar.propTypes = {
 
 export const RichTextEditor = forwardRef(
     (
-        { value, disabled, inputPlaceholder, onChange, errorText, warningText },
+        { value, disabled, inputPlaceholder, onChange, errorText, helpText },
         externalRef
     ) => {
         const [previewMode, setPreviewMode] = useState(false)
         const internalRef = useRef()
         const textareaRef = externalRef || internalRef
-        const validationText = errorText ?? warningText
 
         useEffect(() => textareaRef.current?.focus(), [textareaRef])
 
@@ -237,8 +236,8 @@ export const RichTextEditor = forwardRef(
                 ) : (
                     <Field
                         error={!!errorText}
-                        warning={!errorText && !!warningText}
-                        validationText={validationText}
+                        validationText={errorText}
+                        helpText={helpText}
                     >
                         <UserMentionWrapper
                             onUserSelect={onChange}
@@ -273,6 +272,6 @@ RichTextEditor.propTypes = {
     onChange: PropTypes.func.isRequired,
     disabled: PropTypes.bool,
     errorText: PropTypes.string,
+    helpText: PropTypes.string,
     inputPlaceholder: PropTypes.string,
-    warningText: PropTypes.string,
 }

From ce826696d0e18957ae1ee9a3a02a0eae6f831722 Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Mon, 11 Sep 2023 15:41:17 +0200
Subject: [PATCH 6/9] chore: update pot files

---
 i18n/en.pot | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/i18n/en.pot b/i18n/en.pot
index a7fca475c..0b0491726 100644
--- a/i18n/en.pot
+++ b/i18n/en.pot
@@ -5,8 +5,8 @@ msgstr ""
 "Content-Type: text/plain; charset=utf-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=(n != 1)\n"
-"POT-Creation-Date: 2023-09-05T13:40:21.351Z\n"
-"PO-Revision-Date: 2023-09-05T13:40:21.351Z\n"
+"POT-Creation-Date: 2023-09-11T13:40:55.317Z\n"
+"PO-Revision-Date: 2023-09-11T13:40:55.317Z\n"
 
 msgid "view only"
 msgstr "view only"
@@ -375,13 +375,11 @@ msgid "Write an interpretation"
 msgstr "Write an interpretation"
 
 msgid ""
-"Unable to create a snapshot of this visualization, because it contains no "
-"time dimension. Beware that others viewing this interpretation in the "
-"future may see more data than currently available."
+"Visualization has no time dimension. Other people viewing interpretations "
+"in the future may see more data."
 msgstr ""
-"Unable to create a snapshot of this visualization, because it contains no "
-"time dimension. Beware that others viewing this interpretation in the "
-"future may see more data than currently available."
+"Visualization has no time dimension. Other people viewing interpretations "
+"in the future may see more data."
 
 msgid "Post interpretation"
 msgstr "Post interpretation"

From f38bd378a4d46ef13336ac5c840caf4b60944e3a Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Wed, 13 Sep 2023 09:55:41 +0200
Subject: [PATCH 7/9] fix: default visualizationHasTimeDimension to true

Without this default value the changes in this PR would actually
constitute to a breaking change, at least for the line-listing-app:
if the `visualizationHasTimeDimension` prop is not provided and the
`type` equals `eventVisualization`, the helptext would be rendered.
By defaulting to `true` we avoid this, only explicitly setting
`visualizationHasTimeDimension` to `false` will cause the help text to
show.

I think this is mainly a theoretical issue, because the line-listing-app
will always set the prop. But who knows: there could be a 3rd party app
out there using the analytics library which would break because of this.
---
 .../Interpretations/InterpretationsUnit/InterpretationsUnit.js   | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
index 17359ea4f..8acb12d7b 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationsUnit.js
@@ -187,6 +187,7 @@ InterpretationsUnit.displayName = 'InterpretationsUnit'
 
 InterpretationsUnit.defaultProps = {
     onInterpretationClick: Function.prototype,
+    visualizationHasTimeDimension: true,
 }
 
 InterpretationsUnit.propTypes = {

From 7dd3d310b51f2e800f0aba7c3d506e9a0cc333a7 Mon Sep 17 00:00:00 2001
From: Hendrik de Graaf <HendrikThePendric@users.noreply.github.com>
Date: Thu, 14 Sep 2023 17:18:46 +0200
Subject: [PATCH 8/9] chore: remove stray console.log

Co-authored-by: Edoardo Sabadelli <edoardo@dhis2.org>
---
 .../Interpretations/InterpretationsUnit/InterpretationForm.js    | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
index 27cc2a422..ac3167109 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
@@ -40,7 +40,6 @@ export const InterpretationForm = ({
 
     const inputPlaceholder = i18n.t('Write an interpretation')
 
-    console.log(showNoTimeDimensionHelpText)
     return (
         <MessageEditorContainer
             currentUser={currentUser}

From 813b49dc72a3ef8787011ed04301a75102d39670 Mon Sep 17 00:00:00 2001
From: HendrikThePendric <hendrik@dhis2.org>
Date: Wed, 27 Sep 2023 16:15:50 +0200
Subject: [PATCH 9/9] chore(intepretations): improve no time dimension filter
 warning text

---
 i18n/en.pot                                          | 12 ++++--------
 .../InterpretationsUnit/InterpretationForm.js        |  2 +-
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/i18n/en.pot b/i18n/en.pot
index 0b0491726..bfd246658 100644
--- a/i18n/en.pot
+++ b/i18n/en.pot
@@ -5,8 +5,8 @@ msgstr ""
 "Content-Type: text/plain; charset=utf-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=(n != 1)\n"
-"POT-Creation-Date: 2023-09-11T13:40:55.317Z\n"
-"PO-Revision-Date: 2023-09-11T13:40:55.317Z\n"
+"POT-Creation-Date: 2023-09-27T14:15:13.876Z\n"
+"PO-Revision-Date: 2023-09-27T14:15:13.876Z\n"
 
 msgid "view only"
 msgstr "view only"
@@ -374,12 +374,8 @@ msgstr "Hide interpretation"
 msgid "Write an interpretation"
 msgstr "Write an interpretation"
 
-msgid ""
-"Visualization has no time dimension. Other people viewing interpretations "
-"in the future may see more data."
-msgstr ""
-"Visualization has no time dimension. Other people viewing interpretations "
-"in the future may see more data."
+msgid "Other people viewing this interpretation in the future may see more data."
+msgstr "Other people viewing this interpretation in the future may see more data."
 
 msgid "Post interpretation"
 msgstr "Post interpretation"
diff --git a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
index ac3167109..2b7e9d02c 100644
--- a/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
+++ b/src/components/Interpretations/InterpretationsUnit/InterpretationForm.js
@@ -55,7 +55,7 @@ export const InterpretationForm = ({
                         helpText={
                             showNoTimeDimensionHelpText
                                 ? i18n.t(
-                                      'Visualization has no time dimension. Other people viewing interpretations in the future may see more data.'
+                                      'Other people viewing this interpretation in the future may see more data.'
                                   )
                                 : undefined
                         }