Skip to content

Small fix for grid view#53799

Merged
jscheffl merged 1 commit intoapache:mainfrom
RoyLee1224:task_click
Jul 28, 2025
Merged

Small fix for grid view#53799
jscheffl merged 1 commit intoapache:mainfrom
RoyLee1224:task_click

Conversation

@RoyLee1224
Copy link
Copy Markdown
Contributor

Enhanced the Grid View UX by making the entire row for a Task Group a clickable link.

The onClick handler for the expand/collapse icon now uses event.stopPropagation() to ensure that clicking it only toggles the group's visibility without triggering the parent link's navigation.

Also removed the weird focus outline.

Before After

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Jul 27, 2025
Copy link
Copy Markdown
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!

@jscheffl jscheffl merged commit 34ed71e into apache:main Jul 28, 2025
54 checks passed
@RoyLee1224 RoyLee1224 deleted the task_click branch July 28, 2025 21:14
RoyLee1224 added a commit to RoyLee1224/airflow that referenced this pull request Jul 31, 2025
ferruzzi pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 7, 2025
fweilun pushed a commit to fweilun/airflow that referenced this pull request Aug 11, 2025
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.

2 participants