pierrejeambrun opened a new pull request, #23392:
URL: https://github.com/apache/airflow/pull/23392

   closes: https://github.com/apache/airflow/issues/19943
   
   This is a first draft, let me know what you think.
   
   ## Changes:
   - Filters and Legends Rows are now inside the `Tree` react app and removed 
from the `tree.html` template
   - Added filters on `Dag Run Type` `Run State` and `Task State`.
   - Removed `updated` button, fetched is automatically made on filter change. 
Replaced it with a `Clear Filters` button.
   - Updated colors to light blue for a more unified UI.
   - Removed the initialData, for `useTreeData` query as it populates cache for 
queryKeys until `staleTime` is passed. (successiv refetch were not triggered 
[explanation](https://github.com/tannerlinsley/react-query/discussions/3210#discussioncomment-1990807)
   - Changed the `STATE_COLOR.skipped` -> It was too light, and with the new 
opacity for filtered out task I was hard to tell if it was filtered out or not.
   - Spinner while fetching tree data.
   - A few tests for the custom hook. (I don't know what is the policy here 
about tests for this part of the projet, it seems to have very few tested 
components at the moment).
   
   
   
![image](https://user-images.githubusercontent.com/14861206/166157406-2b407ede-ee2e-4b28-a1ca-c927e175e007.png)
   
![image](https://user-images.githubusercontent.com/14861206/166157437-8f84fc72-f6d9-4ac5-93f0-c349b0699261.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to