bbovenzi commented on code in PR #55604:
URL: https://github.com/apache/airflow/pull/55604#discussion_r2389235120
##########
airflow-core/src/airflow/ui/src/components/LimitedItemsList.tsx:
##########
@@ -69,11 +64,63 @@ export const LimitedItemsList = ({
remainingItems.length === 1 ? (
<Text as="span">{remainingItems[0]}</Text>
) : (
- <Tooltip content={remainingItemsList} interactive={interactive}>
- <Text as="span" cursor="help">
- {translate("limitedList", { count: remainingItems.length })}
- </Text>
- </Tooltip>
+ <Popover.Root lazyMount unmountOnExit>
+ <Popover.Trigger asChild>
+ <Button
+ _hover={{ color: "blue.600", textDecoration: "underline" }}
Review Comment:
Since this is a button now, I think we can remove the custom hover effect
##########
airflow-core/src/airflow/ui/src/components/LimitedItemsList.tsx:
##########
@@ -69,11 +64,63 @@ export const LimitedItemsList = ({
remainingItems.length === 1 ? (
<Text as="span">{remainingItems[0]}</Text>
) : (
- <Tooltip content={remainingItemsList} interactive={interactive}>
- <Text as="span" cursor="help">
- {translate("limitedList", { count: remainingItems.length })}
- </Text>
- </Tooltip>
+ <Popover.Root lazyMount unmountOnExit>
+ <Popover.Trigger asChild>
+ <Button
+ _hover={{ color: "blue.600", textDecoration: "underline" }}
Review Comment:
I thought I had a comment to get rid of this? especially since blue
underline is usually a link
##########
airflow-core/src/airflow/ui/src/components/LimitedItemsList.tsx:
##########
@@ -69,11 +64,63 @@ export const LimitedItemsList = ({
remainingItems.length === 1 ? (
<Text as="span">{remainingItems[0]}</Text>
) : (
- <Tooltip content={remainingItemsList} interactive={interactive}>
- <Text as="span" cursor="help">
- {translate("limitedList", { count: remainingItems.length })}
- </Text>
- </Tooltip>
+ <Popover.Root lazyMount unmountOnExit>
+ <Popover.Trigger asChild>
+ <Button
+ _hover={{ color: "blue.600", textDecoration: "underline" }}
Review Comment:
```suggestion
```
--
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]
