Просмотр исходного кода

Merge pull request #9605 from overleaf/ae-9585-remove-rich-text-survey

Remove Rich Text survey

GitOrigin-RevId: 97003dfb70b8f9cc9118be3f291c7d56d2508f63
Alf Eaton 4 лет назад
Родитель
Сommit
c6f198b3a2

+ 0 - 2
services/web/frontend/js/features/pdf-preview/components/pdf-preview-pane.js

@@ -8,7 +8,6 @@ import { useDetachCompileContext as useCompileContext } from '../../../shared/co
 import FasterCompilesFeedback from './faster-compiles-feedback'
 import { PdfPreviewMessages } from './pdf-preview-messages'
 import CompileTimeWarning from './compile-time-warning'
-import RichTextSurvey from './rich-text-survey'
 
 function PdfPreviewPane() {
   const { pdfUrl } = useCompileContext()
@@ -20,7 +19,6 @@ function PdfPreviewPane() {
       <PdfHybridPreviewToolbar />
       <PdfPreviewMessages>
         <CompileTimeWarning />
-        <RichTextSurvey />
       </PdfPreviewMessages>
       <Suspense fallback={<LoadingPreview />}>
         <div className="pdf-viewer">

+ 0 - 36
services/web/frontend/js/features/pdf-preview/components/rich-text-survey-inner.tsx

@@ -1,36 +0,0 @@
-import { Button } from 'react-bootstrap'
-import { FC, MouseEventHandler } from 'react'
-
-export const RichTextSurveyInner: FC<{
-  handleDismiss: MouseEventHandler<Button>
-  openSurvey: () => void
-}> = ({ handleDismiss, openSurvey }) => (
-  <div className="alert alert-success rich-text-survey-warning" role="alert">
-    <Button
-      className="close"
-      data-dismiss="alert"
-      aria-label="Close"
-      onClick={handleDismiss}
-    >
-      <span aria-hidden="true">&times;</span>
-    </Button>
-    <div className="warning-content">
-      <div>
-        <div className="warning-text">Have you used Rich Text mode?</div>
-        <div className="warning-text">
-          Please participate in our short survey.
-        </div>
-      </div>
-      <div className="upgrade-prompt">
-        <Button
-          type="button"
-          bsSize="sm"
-          bsStyle="primary"
-          onClick={openSurvey}
-        >
-          Take survey
-        </Button>
-      </div>
-    </div>
-  </div>
-)

+ 0 - 59
services/web/frontend/js/features/pdf-preview/components/rich-text-survey.tsx

@@ -1,59 +0,0 @@
-import { FC, memo, useCallback, useEffect, useState } from 'react'
-import usePersistedState from '../../../shared/hooks/use-persisted-state'
-import useScopeValue from '../../../shared/hooks/use-scope-value'
-import { RichTextSurveyInner } from './rich-text-survey-inner'
-
-const SURVEY_URL = 'https://forms.gle/sS4BsUz38GMc81it5'
-const DEFAULT_DELAY = 10 * 1000 // 10 seconds
-
-const RichTextSurvey: FC<{ delay?: number }> = ({ delay = DEFAULT_DELAY }) => {
-  const [dismissed, setDismissed] = usePersistedState(
-    'rich-text-survey-dismissed',
-    false,
-    true
-  )
-
-  const [display, setDisplay] = useState(false)
-
-  const [showRichText] = useScopeValue('editor.showRichText')
-
-  useEffect(() => {
-    let timer: number | undefined
-
-    if (showRichText) {
-      timer = window.setTimeout(() => {
-        setDisplay(true)
-      }, delay)
-    }
-
-    return () => {
-      window.clearTimeout(timer)
-    }
-  }, [showRichText, delay])
-
-  const handleDismiss = useCallback(
-    event => {
-      event.preventDefault()
-      setDismissed(true)
-    },
-    [setDismissed]
-  )
-
-  const openSurvey = useCallback(() => {
-    window.open(SURVEY_URL, '_blank')
-    setDismissed(true)
-  }, [setDismissed])
-
-  if (dismissed || !display) {
-    return null
-  }
-
-  return (
-    <RichTextSurveyInner
-      handleDismiss={handleDismiss}
-      openSurvey={openSurvey}
-    />
-  )
-}
-
-export default memo(RichTextSurvey)

+ 0 - 15
services/web/frontend/stories/pdf-preview-messages.stories.js

@@ -3,9 +3,7 @@ import { useLocalCompileContext } from '../js/shared/context/local-compile-conte
 import { useEffect } from 'react'
 import { PdfPreviewMessages } from '../js/features/pdf-preview/components/pdf-preview-messages'
 import { useScope } from './hooks/use-scope'
-import { RichTextSurveyInner } from '../js/features/pdf-preview/components/rich-text-survey-inner'
 import CompileTimeWarning from '../js/features/pdf-preview/components/compile-time-warning'
-import RichTextSurvey from '../js/features/pdf-preview/components/rich-text-survey'
 
 export default {
   title: 'Editor / PDF Preview / Messages',
@@ -30,20 +28,7 @@ export const Dismissible = () => {
     <div style={{ width: 800, position: 'relative' }}>
       <PdfPreviewMessages>
         <CompileTimeWarning />
-        <RichTextSurvey delay={10} />
       </PdfPreviewMessages>
     </div>
   )
 }
-
-export const Inner = args => {
-  return (
-    <div style={{ width: 800, position: 'relative' }}>
-      <RichTextSurveyInner {...args} />
-    </div>
-  )
-}
-Inner.argTypes = {
-  handleDismiss: { action: 'dismiss' },
-  openSurvey: { action: 'open survey' },
-}