Skip to content

Commit 915692b

Browse files
ref(replays): hide console tab for mobile replays (#69012)
hiding because we decided the console tab isn't useful for video replays; most useful logging context will come from breadcrumbs instead <img width="1046" alt="SCR-20240416-kdcu" src="https://github.com/getsentry/sentry/assets/56095982/b7f41cd2-54a0-4c52-8515-6b2ada4968d1">
1 parent 68ae3ce commit 915692b

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

static/app/views/replays/detail/layout/focusTabs.tsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,14 @@ function getReplayTabs({
2525
// The new trace table inside Breadcrumb items:
2626
const hasTraceTable = organization.features.includes('session-replay-trace-table');
2727

28+
// For video replays, we hide the console, a11y, trace, and memory tabs
29+
// The console tab isn't useful for video replays; most of the useful logging
30+
// context will come from breadcrumbs
31+
// A11y, trace, and memory aren't applicable for mobile
32+
2833
return {
2934
[TabKey.BREADCRUMBS]: t('Breadcrumbs'),
30-
[TabKey.CONSOLE]: t('Console'),
35+
[TabKey.CONSOLE]: isVideoReplay ? null : t('Console'),
3136
[TabKey.NETWORK]: t('Network'),
3237
[TabKey.ERRORS]: t('Errors'),
3338
[TabKey.TRACE]: hasTraceTable || isVideoReplay ? null : t('Trace'),

0 commit comments

Comments
 (0)