[GitHub] [airflow] bbovenzi commented on pull request #31123: UI Add dashboard page

[via GitHub]

bbovenzi commented on PR #31123:
URL: https://github.com/apache/airflow/pull/31123#issuecomment-1538697793

   Great work! Some quick usability comments:
   
   Let's put some labels here:
   <img width="514" alt="Screenshot 2023-05-08 at 12 24 17 PM" 
src="https://user-images.githubusercontent.com/4600967/236877642-907d0404-117d-420e-bac7-89d827aca138.png";>
   
   Looks like we need to fix some overflow:
   <img width="340" alt="Screenshot 2023-05-08 at 12 24 05 PM" 
src="https://user-images.githubusercontent.com/4600967/236877643-9ec79ee3-195f-44e9-b022-a2a1004a2253.png";>
   
   Let's use the same state colors as the rest of the UI. I think that's 
already in your todo though. I also wonder if we can order the legend by number 
of times that the task instance state is in the pie chart:
   <img width="1680" alt="Screenshot 2023-05-08 at 12 23 16 PM" 
src="https://user-images.githubusercontent.com/4600967/236877646-6ed207da-f53a-4be1-b15c-b9831cf55c5d.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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org