nathadfield opened a new pull request, #67543:
URL: https://github.com/apache/airflow/pull/67543

   Collapsed task groups and mapped tasks in the graph and grid views took 
their badge / border / icon colour from the backend's `agg_state()`, which 
ranks queued/scheduled ahead of running/deferred. A group with one queued task 
and five running tasks rendered queued-coloured even though most of its 
children were actively executing. The segmented state bar from #61854 already 
showed the proportional mix correctly, but the surrounding visual elements 
stayed on `agg_state`.
   
   This PR adds a frontend `getDisplayState` helper that ranks active/error 
states ahead of pending states, and routes the dominant colour through it 
across the five rendering surfaces that consume an aggregate state: the graph 
node (`TaskNode.tsx`), grid cell (`GridTI.tsx`), shared tooltip "State:" line, 
group/mapped detail-page headers, and the graph MiniMap. The helper returns 
`null` for no-status dominants so callers keep their existing no-status 
placeholder rather than leaking the serialized `"None"` key into Chakra tokens.
   
   The graph state filter (`useGraphFilteredNodes.ts`) is also realigned, but 
to a "matches any child state" predicate rather than the dominant rendering — 
an operator filtering for "running" expects to find running work even when it's 
buried inside a fail-dominant group. Leaf tasks still match by single state.
   
   Two pre-existing rendering bugs for the `"None"` key in the segmented-bar 
slice and tooltip per-state breakdown are tracked at #67541 with in-code 
pointers at each affected site.
   
   related: #67541
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes
   
   Generated-by: Claude Code (Opus 4.7) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
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]

Reply via email to