alkismavridis commented on code in PR #52765:
URL: https://github.com/apache/airflow/pull/52765#discussion_r2182503892
##########
airflow-core/src/airflow/ui/src/components/LimitedItemsList.tsx:
##########
@@ -67,7 +67,10 @@ export const LimitedItemsList = ({
))}
{shouldTruncate ? (
remainingItems.length === 1 ? (
- <Text as="span">{remainingItems[0]}</Text>
+ <>
+ <Text as="span">{separator}</Text>
Review Comment:
I find it weird that line 64 does not give the separator already. The
condition on line 63 should be true 🤔
When I manager to run it locally I will be able to dive deeper into the
issue.
EDIT:
Is it possible that this is already fixed after 3.0.1? The code seems right.
If so I am really sorry for the trouble and will close this PR.
--
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]
