Skip to content

Conversation

@dheerajturaga
Copy link
Member

@dheerajturaga dheerajturaga commented Oct 26, 2025

The previous bold border didn't provide enough visual distinction for
selected tasks and their connections. This change adds a blue glow effect
and color highlighting to make selected elements stand out clearly.

Grid.View.Glow.mp4
image image

  selected tasks and their connections. This change adds a blue glow effect
  and color highlighting to make selected elements stand out clearly.
@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Oct 26, 2025
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Cool!
As I am not a CSS Expert, letting some other expert another pair of eyes, else LGTM!

Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

Looks really nice, but I'd prefer is someone with better CSS knowledge than me could look more carefully

@bbovenzi
Copy link
Contributor

I get the need to make it more obvious what is selected. I don't know if I agree with the drop shadow effect. Let's just change the border color and width.

@jscheffl
Copy link
Contributor

I get the need to make it more obvious what is selected. I don't know if I agree with the drop shadow effect. Let's just change the border color and width.

Any reason for "don't know if agree"? Just layout nit, tooo fancy? Or any technical reason?

@bbovenzi
Copy link
Contributor

I get the need to make it more obvious what is selected. I don't know if I agree with the drop shadow effect. Let's just change the border color and width.

Any reason for "don't know if agree"? Just layout nit, tooo fancy? Or any technical reason?

Colorful drop shadows just aren't part of our design language, so I would prefer to stay consistent and not add it one-off. Also, I think the border state color and the drop shadow color are clashing.

Perhaps we just override the border color when a task is selected? No information is lost since the state badge is always visible on the task node.

@dheerajturaga
Copy link
Member Author

I get the need to make it more obvious what is selected. I don't know if I agree with the drop shadow effect. Let's just change the border color and width.

Any reason for "don't know if agree"? Just layout nit, tooo fancy? Or any technical reason?

Colorful drop shadows just aren't part of our design language, so I would prefer to stay consistent and not add it one-off. Also, I think the border state color and the drop shadow color are clashing.

Perhaps we just override the border color when a task is selected? No information is lost since the state badge is always visible on the task node.

Done!

image image

@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Oct 30, 2025
@pierrejeambrun pierrejeambrun added this to the Airflow 3.1.2 milestone Oct 30, 2025
@pierrejeambrun pierrejeambrun merged commit 9111175 into apache:main Oct 30, 2025
59 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 30, 2025
…n grid view (#57275)

* The previous bold border didn't provide enough visual distinction for
  selected tasks and their connections. This change adds a blue glow effect
  and color highlighting to make selected elements stand out clearly.

* Brett's suggestions, remove box shadow and override border color when task is selected
(cherry picked from commit 9111175)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

bbovenzi pushed a commit that referenced this pull request Oct 30, 2025
…n grid view (#57275) (#57560)

* The previous bold border didn't provide enough visual distinction for
  selected tasks and their connections. This change adds a blue glow effect
  and color highlighting to make selected elements stand out clearly.

* Brett's suggestions, remove box shadow and override border color when task is selected
(cherry picked from commit 9111175)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
@dheerajturaga dheerajturaga deleted the feature/graph-view-selection-highlight branch October 30, 2025 14:56
@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Nov 10, 2025
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
…pache#57275)

* The previous bold border didn't provide enough visual distinction for
  selected tasks and their connections. This change adds a blue glow effect
  and color highlighting to make selected elements stand out clearly.

* Brett's suggestions, remove box shadow and override border color when task is selected
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. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants