Skip to content

Refactor DataTable to handle internal scrolling and fix overflow on narrow viewports#62603

Merged
vatsrahul1001 merged 1 commit into
apache:mainfrom
choo121600:refactor/data-table
Mar 3, 2026
Merged

Refactor DataTable to handle internal scrolling and fix overflow on narrow viewports#62603
vatsrahul1001 merged 1 commit into
apache:mainfrom
choo121600:refactor/data-table

Conversation

@choo121600

@choo121600 choo121600 commented Feb 28, 2026

Copy link
Copy Markdown
Member

The DataTable component previously rendered a Fragment (<>), delegating scroll behavior to individual consuming pages.
Pages like Connections, Variables, AssetsList, and DagsList wrapped DataTable in <Box overflow="auto">, which caused two issues.

  1. On narrow viewports, horizontal scroll was limited to the table area and content could be clipped.
  2. Vertical scroll moved the entire DataTable (including heading and pagination) instead of just the table body.

This PR moves scroll handling into DataTable itself

Before

Screen.Recording.2026-02-28.at.2.03.12.PM.mov

After

Screen.Recording.2026-02-28.at.2.02.33.PM.mov
Screen.Recording.2026-02-28.at.2.09.51.PM.mov

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    claude

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Feb 28, 2026
@bbovenzi bbovenzi added this to the Airflow 3.1.8 milestone Mar 2, 2026
@vatsrahul1001 vatsrahul1001 merged commit b5eb427 into apache:main Mar 3, 2026
78 checks passed
choo121600 added a commit that referenced this pull request Mar 3, 2026
@vatsrahul1001 vatsrahul1001 added the type:bug-fix Changelog: Bug Fixes label Mar 3, 2026
vatsrahul1001 pushed a commit that referenced this pull request Mar 4, 2026
dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants