Skip to content

fix(ui): restore pointer cursor on task nodes in graph view (#63714)#63810

Closed
Mayankaggarwal8055 wants to merge 1 commit intoapache:v2-11-stablefrom
Mayankaggarwal8055:fix/graph-node-cursor-pointer-2
Closed

fix(ui): restore pointer cursor on task nodes in graph view (#63714)#63810
Mayankaggarwal8055 wants to merge 1 commit intoapache:v2-11-stablefrom
Mayankaggarwal8055:fix/graph-node-cursor-pointer-2

Conversation

@Mayankaggarwal8055
Copy link
Copy Markdown
Contributor

@Mayankaggarwal8055 Mayankaggarwal8055 commented Mar 17, 2026

Fixes incorrect cursor behavior in the Graph view where task nodes do not display a pointer cursor on hover despite being interactive.

Problem

In the Graph view, task nodes are clickable but the cursor remains the default arrow. This creates inconsistent UI feedback and makes interactivity less obvious.

Root Cause

The cursor style for task nodes was either missing or not explicitly set to "pointer" in the node rendering component.

Solution

Explicitly set the cursor style to "pointer" for task nodes in:

airflow/www/static/js/dag/details/graph/DagNode.tsx

Verification

Ran Airflow locally on v2-11-stable

Opened Graph view for a DAG

Confirmed that hovering over task nodes now shows the pointer cursor

Impact

Improves UX by aligning visual feedback with interactivity, making task nodes clearly identifiable as clickable elements.

Fixes #63714

@jroachgolf84
Copy link
Copy Markdown
Collaborator

Can you include a video showing the fix?

@Mayankaggarwal8055
Copy link
Copy Markdown
Contributor Author

Verification

This change has been verified locally:

  • Started Airflow (v2.11.2) using Docker
  • Opened DAG → Graph view
  • Confirmed that task nodes now correctly show a pointer cursor on hover

Demo

video showing the fix:
https://drive.google.com/file/d/13LsLFwpaMXd9-3_VXwvq51GWxvUhwoy8/view?usp=drive_link

@Mayankaggarwal8055 Mayankaggarwal8055 force-pushed the fix/graph-node-cursor-pointer-2 branch from d1ba808 to 738edc0 Compare March 18, 2026 13:08
@Mayankaggarwal8055
Copy link
Copy Markdown
Contributor Author

Hi @jroachgolf84,

I've added a demo video above showing the fix in action. All checks are now passing.

Could you please take another look when you have time?

Thanks!

@pierrejeambrun
Copy link
Copy Markdown
Member

Demo link isn't accessible (permission issue)

@Mayankaggarwal8055
Copy link
Copy Markdown
Contributor Author

Demo link isn't accessible (permission issue)
It is accessible now !

Copy link
Copy Markdown
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

This is not fixing it for me:

Screen.Recording.2026-03-20.at.19.46.02.mov

I think this was intended for groups and while there is no 'TI' and all happens at tasks level

@pierrejeambrun
Copy link
Copy Markdown
Member

The video looks completely unrelated to what we are trying to solve here.

I'm not sure you understand what the issue was about, or how you are fixing it. Most probably AI generated.

Closing.

Feel free to re-open if I missed something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants