This is an automated email from the ASF dual-hosted git repository.

bbovenzi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 0104131f50a update breadcrumb spacing (#47396)
0104131f50a is described below

commit 0104131f50a36f8b9d9cb90926a1a8a0cdcc5f63
Author: Brent Bovenzi <[email protected]>
AuthorDate: Wed Mar 5 12:17:02 2025 -0500

    update breadcrumb spacing (#47396)
---
 airflow/ui/src/components/BreadcrumbStats.tsx    | 2 +-
 airflow/ui/src/components/HeaderCard.tsx         | 2 +-
 airflow/ui/src/layouts/Details/DagBreadcrumb.tsx | 1 -
 airflow/ui/src/layouts/Details/DetailsLayout.tsx | 2 +-
 airflow/ui/src/pages/Asset/Asset.tsx             | 3 +--
 5 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/airflow/ui/src/components/BreadcrumbStats.tsx 
b/airflow/ui/src/components/BreadcrumbStats.tsx
index 5d747363a75..d6fa2066709 100644
--- a/airflow/ui/src/components/BreadcrumbStats.tsx
+++ b/airflow/ui/src/components/BreadcrumbStats.tsx
@@ -33,7 +33,7 @@ export const BreadcrumbStats = ({ links }: { readonly links: 
Links }) => (
         <Stat.Label fontSize="xs" fontWeight="bold">
           {link.title}
         </Stat.Label>
-        <Stat.ValueText fontSize="sm" fontWeight="normal">
+        <Stat.ValueText fontSize="sm" fontWeight="normal" lineHeight={1.5}>
           {index === links.length - 1 ? (
             <Breadcrumb.CurrentLink>
               <HStack>{link.label}</HStack>
diff --git a/airflow/ui/src/components/HeaderCard.tsx 
b/airflow/ui/src/components/HeaderCard.tsx
index fda3ca68f72..56b19ca6878 100644
--- a/airflow/ui/src/components/HeaderCard.tsx
+++ b/airflow/ui/src/components/HeaderCard.tsx
@@ -34,7 +34,7 @@ type Props = {
 };
 
 export const HeaderCard = ({ actions, icon, isRefreshing, state, stats, 
subTitle, title }: Props) => (
-  <Box borderColor="border" borderRadius={8} borderWidth={1} m={2} p={2}>
+  <Box borderColor="border" borderRadius={8} borderWidth={1} ml={2} p={2}>
     <Flex alignItems="center" flexWrap="wrap" justifyContent="space-between" 
mb={2}>
       <Flex alignItems="center" flexWrap="wrap" gap={2}>
         <Heading size="xl">{icon}</Heading>
diff --git a/airflow/ui/src/layouts/Details/DagBreadcrumb.tsx 
b/airflow/ui/src/layouts/Details/DagBreadcrumb.tsx
index 14134668d9b..bdc4659d047 100644
--- a/airflow/ui/src/layouts/Details/DagBreadcrumb.tsx
+++ b/airflow/ui/src/layouts/Details/DagBreadcrumb.tsx
@@ -51,7 +51,6 @@ export const DagBreadcrumb = () => {
 
   const links: Array<{ label: ReactNode | string; labelExtra?: ReactNode; 
title?: string; value?: string }> =
     [
-      { label: "Dags", value: "/dags" },
       {
         label: dag?.dag_display_name ?? dagId,
         labelExtra: (
diff --git a/airflow/ui/src/layouts/Details/DetailsLayout.tsx 
b/airflow/ui/src/layouts/Details/DetailsLayout.tsx
index cbe63d77d4a..8f40dc5498f 100644
--- a/airflow/ui/src/layouts/Details/DetailsLayout.tsx
+++ b/airflow/ui/src/layouts/Details/DetailsLayout.tsx
@@ -59,7 +59,7 @@ export const DetailsLayout = ({ children, error, isLoading, 
tabs }: Props) => {
 
   return (
     <OpenGroupsProvider dagId={dagId}>
-      <HStack justifyContent="space-between">
+      <HStack justifyContent="space-between" mb={2}>
         <DagBreadcrumb />
         <Flex gap={1}>
           <SearchDagsButton />
diff --git a/airflow/ui/src/pages/Asset/Asset.tsx 
b/airflow/ui/src/pages/Asset/Asset.tsx
index 92841f37ed2..2e9385c3e58 100644
--- a/airflow/ui/src/pages/Asset/Asset.tsx
+++ b/airflow/ui/src/pages/Asset/Asset.tsx
@@ -41,7 +41,6 @@ export const Asset = () => {
   );
 
   const links = [
-    { label: "Assets", value: "/assets" },
     {
       label: asset?.name,
       title: "Asset",
@@ -51,7 +50,7 @@ export const Asset = () => {
 
   return (
     <ReactFlowProvider>
-      <HStack justifyContent="space-between">
+      <HStack justifyContent="space-between" mb={2}>
         <BreadcrumbStats links={links} />
       </HStack>
       <ProgressBar size="xs" visibility={Boolean(isLoading) ? "visible" : 
"hidden"} />

Reply via email to