fix(insights): fix ai domain view padding (#79364)

getsentry · Oct 18, 2024 · bd3a215 · bd3a215
1 parent cf4c2cd
commit bd3a215
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 34 deletions.
diff --git a/static/app/views/insights/llmMonitoring/views/llmMonitoringDetailsPage.tsx b/static/app/views/insights/llmMonitoring/views/llmMonitoringDetailsPage.tsx
@@ -147,22 +147,20 @@ export function LLMMonitoringPage({params}: Props) {
         )}
 
         {isInDomainView && (
-          <Layout.Header>
-            <AiHeader
-              headerTitle={
-                <Fragment>
-                  {spanDescription}
-                  <FeatureBadge type={RELEASE_LEVEL} />
-                </Fragment>
-              }
-              breadcrumbs={[
-                {
-                  label: t('Pipeline Summary'),
-                },
-              ]}
-              module={ModuleName.AI}
-            />
-          </Layout.Header>
+          <AiHeader
+            headerTitle={
+              <Fragment>
+                {spanDescription}
+                <FeatureBadge type={RELEASE_LEVEL} />
+              </Fragment>
+            }
+            breadcrumbs={[
+              {
+                label: t('Pipeline Summary'),
+              },
+            ]}
+            module={ModuleName.AI}
+          />
         )}
         <Layout.Body>
           <Layout.Main fullWidth>

diff --git a/static/app/views/insights/llmMonitoring/views/llmMonitoringLandingPage.tsx b/static/app/views/insights/llmMonitoring/views/llmMonitoringLandingPage.tsx
@@ -66,20 +66,18 @@ export function LLMMonitoringPage() {
           </Layout.Header>
         )}
         {isInDomainView && (
-          <Layout.Header>
-            <AiHeader
-              headerTitle={
-                <Fragment>
-                  {MODULE_TITLE}
-                  <PageHeadingQuestionTooltip
-                    title={t('View analytics and information about your AI pipelines')}
-                    docsUrl={MODULE_DOC_LINK}
-                  />
-                </Fragment>
-              }
-              module={ModuleName.AI}
-            />
-          </Layout.Header>
+          <AiHeader
+            headerTitle={
+              <Fragment>
+                {MODULE_TITLE}
+                <PageHeadingQuestionTooltip
+                  title={t('View analytics and information about your AI pipelines')}
+                  docsUrl={MODULE_DOC_LINK}
+                />
+              </Fragment>
+            }
+            module={ModuleName.AI}
+          />
         )}
         <Layout.Body>
           <Layout.Main fullWidth>

diff --git a/static/app/views/insights/pages/ai/aiOverviewPage.tsx b/static/app/views/insights/pages/ai/aiOverviewPage.tsx
@@ -136,9 +136,7 @@ function AiOverviewPage() {
       organization={organization}
       renderDisabled={NoAccess}
     >
-      <Layout.Header>
-        <AiHeader headerTitle={AI_LANDING_TITLE} />
-      </Layout.Header>
+      <AiHeader headerTitle={AI_LANDING_TITLE} />
       <Layout.Body>
         <Layout.Main fullWidth>
           <ModuleLayout.Layout>

diff --git a/static/app/views/insights/pages/ai/aiPageHeader.tsx b/static/app/views/insights/pages/ai/aiPageHeader.tsx
@@ -18,7 +18,6 @@ type Props = {
   module?: HeaderProps['selectedModule'];
 };
 
-// TODO - add props to append to breadcrumbs and change title
 export function AiHeader({module, headerTitle, headerActions, breadcrumbs}: Props) {
   const {slug} = useOrganization();