Skip to content

Conversation

@bbovenzi
Copy link
Contributor

@bbovenzi bbovenzi commented Feb 3, 2023

In 2.5.0, I set the max height for the grid view to be too restricted which hurt UX. This PR basically doubles the maxHeight to allow a user to see more of the grid at once but without scrolling the entire page so far as to lose context.

Fixes: #29100

Before:
Feb-03-2023 15-47-04

After:
Feb-03-2023 15-46-42


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Feb 3, 2023
@bbovenzi bbovenzi added this to the Airflow 2.5.2 milestone Feb 3, 2023
@jedcunningham jedcunningham merged commit 643d736 into apache:main Feb 5, 2023
@jedcunningham jedcunningham deleted the calculate-new-max-grid-heights branch February 5, 2023 23:15
@pierrejeambrun pierrejeambrun added the type:misc/internal Changelog: Misc changes that should appear in change log label Feb 27, 2023
pierrejeambrun pushed a commit that referenced this pull request Mar 7, 2023
pierrejeambrun pushed a commit that referenced this pull request Mar 8, 2023
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. area:webserver Webserver related Issues type:misc/internal Changelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unnecessary scrollbars in grid view

4 participants