Add filtering for tasks view

[dstandish]

Closes #53048

Adds some filters in the tasks view.

I elected not to add a new endpoint but just to filter what was there as a good first improvement. I think adding the endpoint introduces meaningful complexity and some questions and ambiguities that would need to be resolved.

I also didn't add the time-based filtering for the same reason. It's not how the current endpoint works, and I think adding time-based filtering would require more refactoring and more thought about the interface.

All other filters in the ticket, however, I have added.

update

I added basic task-name search filter and refactored the filters into a separate component like done with DagsList

tasks.filters.mov

[bbovenzi]

Cool!

Left a few comments. This was always an awkward page. I think we should also show more Task info on the page. Like the card view doesn't show Retry or Mapped info. Is there anything else you would want to see here? I'd imagine a search by task_id would be very valuable!

[pierrejeambrun]

Nice!

Just a few comments:
We need some margin between the two rows

'reset' filter button doesn't seem to reset the search field

When filling multiple values, it doesn't display well:
https://github.com/user-attachments/assets/b138ad6a-5374-470d-9ee4-da2543488dab

[dstandish]

Nice!

Just a few comments: We need some margin between the two rows

'reset' filter button doesn't seem to reset the search field

When filling multiple values, it doesn't display well: https://github.com/user-attachments/assets/b138ad6a-5374-470d-9ee4-da2543488dab

I don't really know how to fix these things quickly and I need to move on to asset partitioning unfortunately. Fancy adding to this PR?

[dstandish]

ok i fixed most of the things

the search bar thing ---- basically, it has internal calls to useState, so it cannot be driven exclusively by changes to searchParams

if we're always going to use searchParams, probably it should not use its own state store -- seems like if you do that then the only way to clear the search bar is to click the X

i also didn't mess with the filter court -- this could be done as followup i suppose

i added a justify to improve the horizontal spacing . it's not perfect but it's better. if you have fixes feel free to push em. otherwise maybe we can just merge this and anyone with time can tweak and refine?

[dstandish]

alternatively i can just remove the text search -- it was not part of the original ticket

[waqas-saeed]

Hi @dstandish in your video I can see this toggle button in the middle in black colour. it looks like you can use it to hide details page on the right. is that correct?
which airflow version are you using for this? I'm running 3.0.4 and I can't see it in my airflow

[pierrejeambrun]

As discussed, don't mind the search bar reset value as this is not related to this PR but to how search bar work, we need to solve it in a follow up.

[dstandish]

As discussed, don't my the search bar reset value as this is not related to this PR but to how search bar work, we need to solve it in a follow up.

Yeah I noticed also with dags list, the reset button does not reset the search filters.

Let me know if there's anything further you need me to change in this one.

Thanks

[dstandish]

hey @bbovenzi @pierrejeambrun if y'all want to take another pass and see if there's anything left to do so we can close this out?

[bbovenzi]

made some small changes. now it looks good

[dstandish]

made some small changes. now it looks good

thanks @bbovenzi