bbovenzi commented on PR #57680: URL: https://github.com/apache/airflow/pull/57680#issuecomment-3613657727
> The only one that was not straightforward was the DagsList because it places the ToggleTableDisplay component between the row count heading and table (The screenshot shows how it shifted to above the row count heading). Since this component is only used on the DagsList page, does it make sense to support that layout through moving that toggle to an optional prop on the DataTable or should it be slotted in by the caller? Open to other approaches as well. Great work. Let's make it an optional prop so that we're not creating excess vertical space on the dags list page. Maybe in the future other table views will have card list variants. -- 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]
