parkhojeong commented on PR #68346: URL: https://github.com/apache/airflow/pull/68346#issuecomment-4710654425
> Nice work - I just tested from UX perspective (not reviewing code) and I like it. Just one nit: Compared to the rest of UI and other modals the font pitch is lower than in other areas. Font sizes should be adjusted to other UI standards. Thanks! Adjusted the sizes to match the UI standards and simplified the props. https://github.com/apache/airflow/pull/68346/commits/2f7125501735d10143cc7534da5c849939eaade7 https://github.com/apache/airflow/pull/68346/commits/20a0db637db69df8f87203381f070ccddf7a99db | Another Modal | Another Table | HITL Modal | HITL Drawer | | --- | --- | --- | --- | | <img width="1437" height="560" alt="image" src="https://github.com/user-attachments/assets/ca6fc99c-5c21-46a7-9c37-4c149db5ab7c"; /> | <img width="1670" height="548" alt="image" src="https://github.com/user-attachments/assets/5af03f61-5e70-4f0e-866a-16652dd60667"; /> | <img width="1654" height="995" alt="image" src="https://github.com/user-attachments/assets/d8926231-b6b0-4ad4-8ec0-10c6dd8cc56d"; /> | <img width="968" height="820" alt="image" src="https://github.com/user-attachments/assets/971f12b8-9374-42f1-bdd6-7b4bb385f5b9"; /> | -- 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]
