Browse Source

fix(replays): Expand on why Memory metrics might not be captured (#40425)

Slight design change, with fonts and stuff, and new copy:

<img width="786" alt="Screen Shot 2022-10-21 at 2 35 46 PM"
src="https://user-images.githubusercontent.com/187460/197292418-3e0ab53f-1cfb-4440-bde5-2ecd13eeb7a2.png">

Fixes #40407
Ryan Albrecht 2 years ago
parent
commit
5a8c2b83f1
1 changed files with 7 additions and 4 deletions
  1. 7 4
      static/app/views/replays/detail/memoryChart.tsx

+ 7 - 4
static/app/views/replays/detail/memoryChart.tsx

@@ -8,7 +8,7 @@ import Grid from 'sentry/components/charts/components/grid';
 import Tooltip from 'sentry/components/charts/components/tooltip';
 import XAxis from 'sentry/components/charts/components/xAxis';
 import YAxis from 'sentry/components/charts/components/yAxis';
-import EmptyStateWarning from 'sentry/components/emptyStateWarning';
+import EmptyMessage from 'sentry/components/emptyMessage';
 import {showPlayerTime} from 'sentry/components/replays/utils';
 import {t} from 'sentry/locale';
 import space from 'sentry/styles/space';
@@ -42,9 +42,12 @@ function MemoryChart({
 
   if (memorySpans.length <= 0) {
     return (
-      <EmptyStateWarning withIcon={false} small>
-        {t('No memory metrics found')}
-      </EmptyStateWarning>
+      <EmptyMessage
+        title={t('No memory metrics found')}
+        description={t(
+          'Memory metrics are only captured within Chromium based browser sessions.'
+        )}
+      />
     );
   }