-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Improve Dags Filter UI #60346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve Dags Filter UI #60346
Conversation
choo121600
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
airflow-core/src/airflow/ui/src/components/DataTable/ToggleTableDisplay.tsx
Outdated
Show resolved
Hide resolved
Backport failed to create: v3-1-test. View the failure log Run details
You can attempt to backport this manually by running: cherry_picker 5cadabe v3-1-testThis should apply the commit to the v3-1-test branch and leave the commit in conflict state marking After you have resolved the conflicts, you can continue the backport process by running: cherry_picker --continue |
* Fix colors and remove reset all * Create ButtonToggle component * Remove manual bg setting (cherry picked from commit 5cadabe)
* Fix colors and remove reset all * Create ButtonToggle component * Remove manual bg setting (cherry picked from commit 5cadabe)
* Fix colors and remove reset all * Create ButtonToggle component * Remove manual bg setting

Follow up from #60298
Make a single ButtonGroupToggle component we can use in many place
Make the DagsFilters component wrap on smaller screens
Changed the selected colors to use
colorPalette.solidto make it more visible on dark and light modes.Moved the Required Actions as a separate on/off button because it isn't actually filtering by state
Removed Reset Filters button. I feel like with the filters being more noticeable that a button showing up and changing the page's format got in the way more than it helps.
Was generative AI tooling used to co-author this PR?
Generated-by: [Cursor] following the guidelines
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.