Skip to content

Reduce DAG card tooltip controller overhead - #69681

Merged
potiuk merged 2 commits into
apache:mainfrom
shivaam:codex/reduce-dag-card-tooltip-overhead
Jul 12, 2026
Merged

Reduce DAG card tooltip controller overhead#69681
potiuk merged 2 commits into
apache:mainfrom
shivaam:codex/reduce-dag-card-tooltip-overhead

Conversation

@shivaam

@shivaam shivaam commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Card view requests up to 14 recent runs per DAG. The first 50 cards can therefore create up to 700 independent Chakra tooltip roots and state machines, even though only one tooltip per card can be active at a time.

Reuse one multi-trigger tooltip root per DAG card. Keep every recent-run bar visible and clickable, and resolve the selected run's existing tooltip details through Tooltip.Context when the user focuses or hovers a trigger.

I tested it locally and it feels slightly faster.

No visual change is expected.

related: #69609

Testing:

  • pnpm exec vitest run src/pages/DagsList/DagCard.test.tsx
  • pnpm exec tsc --p tsconfig.app.json --noEmit
  • pnpm exec eslint src/pages/DagsList/RecentRuns.tsx src/pages/DagsList/DagCard.test.tsx
  • pnpm build
  • uvx prek run --files airflow-core/src/airflow/ui/src/pages/DagsList/RecentRuns.tsx airflow-core/src/airflow/ui/src/pages/DagsList/DagCard.test.tsx

Was generative AI tooling used to co-author this PR?
  • Yes (Codex)

Generated-by: Codex following the guidelines

UI-fix1.mov

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Jul 9, 2026
@shivaam
shivaam marked this pull request as ready for review July 9, 2026 23:46

@shahar1 shahar1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jul 11, 2026
@potiuk

potiuk commented Jul 12, 2026

Copy link
Copy Markdown
Member

Nice!

@potiuk
potiuk merged commit a0e0ae1 into apache:main Jul 12, 2026
88 checks passed
joshuabvarghese pushed a commit to joshuabvarghese/airflow that referenced this pull request Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants