antonio-mello-ai commented on PR #62835:
URL: https://github.com/apache/airflow/pull/62835#issuecomment-4026343034
Thanks for the review @pierrejeambrun!
**CI fix:** Rebased on latest `main` and applied the formatting fixes
(import ordering + line wrapping) that `ts-compile-lint-ui` was catching.
Should pass now.
**Before vs After — what this PR changes:**
1. **X-axis labels**: Previously used raw `DEFAULT_DATETIME_FORMAT`
timestamps which took significant space. Now uses `formatDate()` with the
user's selected timezone and an adaptive format:
- Data spanning < 1 day: shows `HH:mm:ss`
- Data spanning >= 1 day: shows `MMM DD HH:mm`
2. **Animation during auto-refresh**: Previously the chart re-animated on
every auto-refresh poll, causing visual flickering. Now disables animation when
auto-refresh is active and a run is in a pending state.
I don't have a local Airflow environment to capture screenshots at the
moment. If you'd like I can set one up and provide a recording, or you can
verify by running the UI locally with a DAG that has recent runs — the changes
are visible on the Dag Overview page's duration chart.
--
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]
