eddiesr93 commented on PR #70142: URL: https://github.com/apache/airflow/pull/70142#issuecomment-5026315266
Pushed a follow-up commit after re-checking the layout against the `AssetEvents` sibling on the Dag overview. Freeing the card width handed the chart height to Chart.js' default 2:1 aspect ratio, which the old fixed `350px` had been capping as a side effect. Measured on a 1400px row: the canvas came out **992×496**, and on a 2560px monitor it would have scaled past 1000px tall. The chart now renders into a fixed-height box with `maintainAspectRatio: false`, so it is 280px tall at every width, and the loading skeletons were bumped from 200px to match so there is no jump on load. For the record, the sibling panel is unaffected — `AssetEvents` measures 380px wide with the old fixed width, with the flexed card, and with a capped variant, because it sizes to its content and there is spare room in the row either way. The screenshots above have been regenerated against the fixed height. --- Drafted-by: Claude Code (Opus 4.8) (no human review before posting) -- 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]
