kevinhongzl commented on PR #74883: URL: https://github.com/apache/airflow/pull/74883#issuecomment-6286747849
Thanks for the feedback @bbovenzi ! I took another look at how notes are displayed in the airflow UI and came up with two possible designs ### Design 1: Stacked cards for task states The front card shows the primary state of the mapped tasks, while the back card shows the secondary state. If all mapped tasks share the same state, the grid cell remains unstacked. <img width="268.5" height="85" alt="image" src="https://github.com/user-attachments/assets/72bb0063-54e0-4d09-aeae-5842174a69fb"; /> <img width="267" height="87.5" alt="image" src="https://github.com/user-attachments/assets/2fd0b2dd-5dff-4721-89f1-e6e6ef96f569"; /> ### Design 2: Diagonal split for task states + badge for notes I noticed the current note indicator isn't very noticeable in dark mode, and it takes up the upper triangle just to show whether a task has a note. The space could be used to represent different combinations of task states instead. So I was thinking we could move the note indicators to a small badge and use the diagonal split for primary and secondary states. This would also involve changing how notes are displayed, though, which is a bit outside the scope of this PR. <img width="274.5" height="82.5" alt="image" src="https://github.com/user-attachments/assets/0c6e9bb9-6dc8-4f6b-af2f-988eaf1dd608"; /> <img width="263.5" height="83.5" alt="image" src="https://github.com/user-attachments/assets/763162d4-5bfa-478e-9f76-a9681b66ebfd"; /> <br><br> What do you think? -- 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]
