BrianLii commented on code in PR #55652:
URL: https://github.com/apache/airflow/pull/55652#discussion_r2347656974
##########
airflow-core/src/airflow/ui/src/components/HeaderCard.tsx:
##########
@@ -38,7 +38,7 @@ export const HeaderCard = ({ actions, icon, isRefreshing,
state, stats, subTitle
const { t: translate } = useTranslation();
return (
- <Box borderColor="border.emphasized" borderRadius={8} borderWidth={1}
ml={2} p={2}>
+ <Box borderColor="border.emphasized" borderRadius={8} borderWidth={1}
p={2}>
Review Comment:
Thanks for asking!
Do you mean adding a margin to the Asset Event list to align it with the
header? Currently, the map on the left-hand side already applies a right
padding. Adding extra margin would make the horizontal spacing larger than the
vertical spacing between the header and the list card. I think it's better to
keep all margins consistent by removing the additional margin from the header
card instead.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
