Skip to content

Fix cursor not showing pointer on task nodes in graph view (#63714) #63735

Closed
shashbha14 wants to merge 1 commit intoapache:mainfrom
shashbha14:fix-graph-view-cursor-pointer-63714
Closed

Fix cursor not showing pointer on task nodes in graph view (#63714) #63735
shashbha14 wants to merge 1 commit intoapache:mainfrom
shashbha14:fix-graph-view-cursor-pointer-63714

Conversation

@shashbha14
Copy link
Contributor

Fixes a regression where the cursor no longer changed to a pointer when hovering over task nodes in the Graph view.
Changes

  • TaskNode.tsx: Changed cursor="default" back to cursor="pointer" on the task node Flex wrapper
    Why ?
    The pointer cursor is an important UX signal that the task nodes are clickable. This had been set to default in a previous change.

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Mar 16, 2026
@bbovenzi
Copy link
Contributor

Do you know what caused this regression? Because I am also unable to replicate this.

@potiuk
Copy link
Member

potiuk commented Mar 17, 2026

@shashbha14 This PR has a few issues that need to be addressed before it can be reviewed — please see our Pull Request quality criteria.

Issues found:

  • Other failing CI checks: Failing: check-newsfragment-pr-number. Run prek run --from-ref main locally to reproduce. See static checks docs.

Note: Your branch is 343 commits behind main. Some check failures may be caused by changes in the base branch rather than by your PR. Please rebase your branch and push again to get up-to-date CI results.

What to do next:

  • The comment informs you what you need to do.
  • Fix each issue, then mark the PR as "Ready for review" in the GitHub UI - but only after making sure that all the issues are fixed.
  • There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates.
  • Maintainers will then proceed with a normal review.

There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates. If you have questions, feel free to ask on the Airflow Slack.

Copy link
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.

Closing.

That's not reproducible and most likely an AI generated fix that wasn't tested first. (targetting main 3.x, while original issue which isn't linked btw is for 2.11... #63714)

Similarly to the dozen of PR closed from this author closing this one.
https://github.com/apache/airflow/pulls/@shashbha14

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

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants