pierrejeambrun commented on code in PR #67879:
URL: https://github.com/apache/airflow/pull/67879#discussion_r3340439875
##########
airflow-core/src/airflow/ui/src/pages/Dag/Calendar/CalendarCell.tsx:
##########
@@ -51,6 +51,18 @@ export const CalendarCell = ({
const hasData = Boolean(cellData && relevantCount > 0);
const hasTooltip = Boolean(cellData);
+ // States present in this cell, computed with the same view-mode-aware logic
the
+ // tooltip uses (see CalendarTooltip). Exposed as a `data-states` attribute
so e2e
+ // tests can read run states from the DOM instead of hovering — the tooltip
does
+ // not open reliably under synthetic pointer events in headless Firefox.
Review Comment:
nit: We could make these two comments less verbose.
--
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]
