From 9b818076ae8d10a187362925c9b6a3da822d3151 Mon Sep 17 00:00:00 2001
From: Scott Cooper <scttcper@gmail.com>
Date: Wed, 16 Oct 2024 15:38:57 -0700
Subject: [PATCH 1/2] fix(issues): Remove highlights all tags link

Removes the "view all issue tags" from the highlights section for streamline
---
 .../highlights/highlightsDataSection.tsx      | 41 ++++++-------------
 .../streamline/eventNavigation.tsx            |  2 +-
 2 files changed, 14 insertions(+), 29 deletions(-)

diff --git a/static/app/components/events/highlights/highlightsDataSection.tsx b/static/app/components/events/highlights/highlightsDataSection.tsx
index 8238d1294cb4f..f27866b247ae4 100644
--- a/static/app/components/events/highlights/highlightsDataSection.tsx
+++ b/static/app/components/events/highlights/highlightsDataSection.tsx
@@ -4,7 +4,7 @@ import styled from '@emotion/styled';
 
 import {openModal} from 'sentry/actionCreators/modal';
 import {hasEveryAccess} from 'sentry/components/acl/access';
-import {Button, LinkButton} from 'sentry/components/button';
+import {Button} from 'sentry/components/button';
 import ButtonBar from 'sentry/components/buttonBar';
 import ErrorBoundary from 'sentry/components/errorBoundary';
 import {ContextCardContent} from 'sentry/components/events/contexts/contextCard';
@@ -37,8 +37,6 @@ import {useLocation} from 'sentry/utils/useLocation';
 import useOrganization from 'sentry/utils/useOrganization';
 import {SectionKey} from 'sentry/views/issueDetails/streamline/context';
 import {InterimSection} from 'sentry/views/issueDetails/streamline/interimSection';
-import {Tab, TabPaths} from 'sentry/views/issueDetails/types';
-import {useGroupDetailsRoute} from 'sentry/views/issueDetails/useGroupDetailsRoute';
 import {useHasStreamlinedUI} from 'sentry/views/issueDetails/utils';
 
 interface HighlightsDataSectionProps {
@@ -259,32 +257,19 @@ export default function HighlightsDataSection({
 }: HighlightsDataSectionProps) {
   const organization = useOrganization();
   const hasStreamlinedUI = useHasStreamlinedUI();
-  const location = useLocation();
-  const {baseUrl} = useGroupDetailsRoute();
 
-  const viewAllButton = hasStreamlinedUI ? (
-    // Streamline details ui has "Jump to" feature, instead we'll show the drawer button
-    <LinkButton
-      to={{
-        pathname: `${baseUrl}${TabPaths[Tab.TAGS]}`,
-        query: location.query,
-        replace: true,
-      }}
-      size="xs"
-    >
-      {t('View All Issue Tags')}
-    </LinkButton>
-  ) : viewAllRef ? (
-    <Button
-      onClick={() => {
-        trackAnalytics('highlights.issue_details.view_all_clicked', {organization});
-        viewAllRef?.current?.scrollIntoView({behavior: 'smooth'});
-      }}
-      size="xs"
-    >
-      {t('View All')}
-    </Button>
-  ) : null;
+  const viewAllButton =
+    !hasStreamlinedUI && viewAllRef ? (
+      <Button
+        onClick={() => {
+          trackAnalytics('highlights.issue_details.view_all_clicked', {organization});
+          viewAllRef?.current?.scrollIntoView({behavior: 'smooth'});
+        }}
+        size="xs"
+      >
+        {t('View All')}
+      </Button>
+    ) : null;
 
   return (
     <InterimSection
diff --git a/static/app/views/issueDetails/streamline/eventNavigation.tsx b/static/app/views/issueDetails/streamline/eventNavigation.tsx
index 2e9e040c4d7a8..1bf953d2c2608 100644
--- a/static/app/views/issueDetails/streamline/eventNavigation.tsx
+++ b/static/app/views/issueDetails/streamline/eventNavigation.tsx
@@ -76,7 +76,7 @@ const EventNavOrder = [
 ];
 
 const sectionLabels = {
-  [SectionKey.HIGHLIGHTS]: t('Event Highlights'),
+  [SectionKey.HIGHLIGHTS]: t('Highlights'),
   [SectionKey.STACKTRACE]: t('Stack Trace'),
   [SectionKey.TRACE]: t('Trace'),
   [SectionKey.EXCEPTION]: t('Stack Trace'),

From c98f85b88b5c5ac575e4764f0ee2a861841a7ba1 Mon Sep 17 00:00:00 2001
From: Scott Cooper <scttcper@gmail.com>
Date: Wed, 16 Oct 2024 15:48:17 -0700
Subject: [PATCH 2/2] update test copy

---
 .../views/issueDetails/streamline/eventNavigation.spec.tsx    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/static/app/views/issueDetails/streamline/eventNavigation.spec.tsx b/static/app/views/issueDetails/streamline/eventNavigation.spec.tsx
index 1f067e81471f5..8a5ac19920e0d 100644
--- a/static/app/views/issueDetails/streamline/eventNavigation.spec.tsx
+++ b/static/app/views/issueDetails/streamline/eventNavigation.spec.tsx
@@ -128,13 +128,13 @@ describe('EventNavigation', () => {
     expect(screen.queryByText('Jump To:')).not.toBeInTheDocument();
     expect(screen.queryByText('Replay')).not.toBeInTheDocument();
     expect(screen.queryByText('Tags')).not.toBeInTheDocument();
-    expect(screen.queryByText('Event Highlights')).not.toBeInTheDocument();
+    expect(screen.queryByText('Highlights')).not.toBeInTheDocument();
   });
 
   it('does show jump to sections when the sections render', () => {
     render(<EventNavigation {...defaultProps} />);
     expect(screen.getByText('Jump to:')).toBeInTheDocument();
-    expect(screen.getByText('Event Highlights')).toBeInTheDocument();
+    expect(screen.getByText('Highlights')).toBeInTheDocument();
     expect(screen.getByText('Replay')).toBeInTheDocument();
     expect(screen.getByText('Tags')).toBeInTheDocument();
   });